-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[General] Update LMGTFY link #6255
Conversation
The old URL is still technically working, but it's certificates have expired which makes it very hard to actually use the site now. The The new site redirects to |
Tell me what's the taken decision and I'll make the changes. 👍 |
I think it might be a good idea to just cut the middle man entirely and link straight to google instead. |
Yeah, I'd agree that's a good compromise. |
I don't agree with cutting the middle-man since it doesn't quite have the same power as "let me google that for you" animation but the https://lmgtfy2.com seems to work nicely? Not sure if we want the variant with "Internet Explainer", I kinda like it but it may be too much lol |
Sure it doesn't have the same power but is it really worth keeping around for a silly joke? I think cutting the middle-man also helps prevent potential abuse. For example what if whatever site we choose to use decides to let their domain lapse and someone else buys it to exploit anyone using that service? That's outside our control and leads to us having to find yet another alternative like we are now. I think the best approach would be just to leave the joke in the command name and skip this hassle in the future. |
Absolutely. However, we could also expand the command to do the explainer on its own :) |
I understand both point of view. However, that would totally beat the command's joke to directly use Google's website. That's -1 from me. It's Also agree that I'll update the branch to use Jacknmen's host, but I'm still listening to talk regarding this change. Also, the command explanation required. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good enough for now, we can readdress this if (when) this site goes down.
happened to come across https://lmgt.org/ also just now browsing reddit which seems to also work btw |
I've come across this project and has a demo we could use, which, doesn't track you, and actually redirects you to Google. https://github.com/NatoBoram/lmgtfy https://natoboram.github.io/lmgtfy/search?q=Red-DiscordBot&btnK=Google+Search https://github.com/Cog-Creators/Red-DiscordBot/pull/6007/commits |
Installed a test version import urllib.parse
search_string = "What is Red-DiscordBot"
encoded_search_string = urllib.parse.quote(search_string)
link = f"[Here's your link](https://lmgtfy.lena.moe/lmgtfy/search?q={encoded_search_string}&btnK=Google+Search)"
await ctx.send(link) |
You should consider making a new issue/create a new PR to consider changing the modification made from this PR. |
Description of the changes
This PR propose to change the LMGTFY link from
https://lmgtfy.app
tohttps://letmegooglethat.com
as it seems that the old domain seems to experience problems that will never be resolved.It's been weeks since the HSTS error has been happening.
Also remove the
s
query parameter, it seems unused.Have the changes in this PR been tested?
No, but should be fine.