Skip to content
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

Merged
merged 3 commits into from Nov 6, 2023
Merged

[General] Update LMGTFY link #6255

merged 3 commits into from Nov 6, 2023

Conversation

madebylydia
Copy link
Contributor

Description of the changes

This PR propose to change the LMGTFY link from https://lmgtfy.app to https://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.

image

Also remove the s query parameter, it seems unused.

Have the changes in this PR been tested?

No, but should be fine.

@github-actions github-actions bot added the Category: Cogs - General This is related to the General cog. label Oct 7, 2023
@Flame442
Copy link
Member

Flame442 commented Oct 7, 2023

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 s query param is used by that site to specify it should redirect to a google search, rather than some other search engine. The site you have changed it to may not support that option.

The new site redirects to gprivate.com instead of google.com. Considering it has an about page that is unfilled out, a merch page underneath, and redirects to a non-google site, I'm hesitant to say this site is entirely safe. Considering the history of this command, and the fact that there doesn't seem like a great replacement site, this might be time to just cut the command.

@madebylydia
Copy link
Contributor Author

Tell me what's the taken decision and I'll make the changes. 👍

@TrustyJAID
Copy link
Member

I think it might be a good idea to just cut the middle man entirely and link straight to google instead. https://www.google.com/search?q={search_term} that way there's no issues and we maintain the functionality.

@Flame442
Copy link
Member

Flame442 commented Oct 7, 2023

Yeah, I'd agree that's a good compromise.

@Jackenmen
Copy link
Member

Jackenmen commented Oct 7, 2023

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 letmegooglethat.com site (or googlethatforyou.com which is really the same thing) is not an acceptable choice either...

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

@Jackenmen Jackenmen added the Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. label Oct 7, 2023
@TrustyJAID
Copy link
Member

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.

@Jackenmen
Copy link
Member

but is it really worth keeping around for a silly joke?

Absolutely.

However, we could also expand the command to do the explainer on its own :)

@madebylydia
Copy link
Contributor Author

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 [p]lmgtfy, not [p]google. 😉

Also agree that lmgtfy2.com seems like the best option for now, but I also have little doubt to the website's future considering the copyright says "2022-2023", to take back on what @TrustyJAID said about website's availability.
I don't think it's really a bad thing nor a big time-consuming task to find another alternative to the LMGTFY provider, if needed.

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.

@Flame442 Flame442 added the hacktoberfest-accepted Used to mark a PR as valid Hacktoberfest contribution. DO NOT REMOVE UNTIL END OF NOVEMBER (sic!)! label Oct 7, 2023
@Flame442 Flame442 self-assigned this Nov 6, 2023
Copy link
Member

@Flame442 Flame442 left a 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.

@Flame442 Flame442 merged commit 6c4a5e5 into Cog-Creators:V3/develop Nov 6, 2023
18 checks passed
@red-githubbot red-githubbot bot added the Changelog Entry: Pending Changelog entry for this PR hasn't been added by repo maintainers yet. label Nov 6, 2023
@Flame442 Flame442 added this to the 3.5.6 milestone Nov 6, 2023
@BenCos17
Copy link

BenCos17 commented Nov 8, 2023

happened to come across https://lmgt.org/ also just now browsing reddit which seems to also work btw

@DJTOMATO
Copy link
Contributor

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

@DJTOMATO
Copy link
Contributor

DJTOMATO commented Nov 15, 2023

Installed a test version
https://lmgtfy.lena.moe

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)

@madebylydia
Copy link
Contributor Author

You should consider making a new issue/create a new PR to consider changing the modification made from this PR.

@Jackenmen Jackenmen added Changelog Entry: Skipped Changelog entry for this PR is unnecessary. and removed Changelog Entry: Pending Changelog entry for this PR hasn't been added by repo maintainers yet. labels Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Cogs - General This is related to the General cog. Changelog Entry: Skipped Changelog entry for this PR is unnecessary. hacktoberfest-accepted Used to mark a PR as valid Hacktoberfest contribution. DO NOT REMOVE UNTIL END OF NOVEMBER (sic!)! Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants