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

[GiphyBridge] use not rate limited public api key (error 403 fix) #2625

Merged
merged 1 commit into from
Apr 10, 2022

Conversation

User123698745
Copy link
Contributor

Since more than a week the Giphy bridge stopped working.
The bridge constantly returns 403 (e.g: https://bridge.easter.fr/?action=display&bridge=Giphy&s=%40fallguysgame&format=Json).
Calls to the Giphy API with the currently used public beta key constantly return 403 BANNED (e.g: https://api.giphy.com/v1/stickers/search?api_key=dc6zaTOxFJmzC&q=%40FallGuysGame).
This is probably due to the severe rate limiting of beta api keys.

I looked at the network traffic of Giphy.com and found that the website calls the API with this key: Gc7131jiJuvI7IdN0HZ1D7nh0ow5BU6g
It's probably Giphy's own prod api key and therefore not rate limited. From a quick google search it looks likes others already use that key and that it is around since more than a year.

This PR changes the public beta key "dc6zaTOxFJmzC" to Giphy's own prod api key "Gc7131jiJuvI7IdN0HZ1D7nh0ow5BU6g".
I tested this change on a private instance for couple of days and have not noticed any problem.

@github-actions
Copy link

github-actions bot commented Apr 9, 2022

Pull request artifacts

file last change
Giphy-current-context1 2022-04-09, 23:54:05
Giphy-pr-context1 2022-04-09, 23:54:05

@dvikan dvikan merged commit 722f9ff into RSS-Bridge:master Apr 10, 2022
@dvikan
Copy link
Contributor

dvikan commented Apr 10, 2022

Thanks.
Also related: #2616

@User123698745 User123698745 deleted the giphy-api-key branch April 10, 2022 12:13
@Binnette
Copy link
Contributor

Thanks ❤️ Giphy Bridge is now working again ❤️

Kwbmm pushed a commit to Kwbmm/rss-bridge that referenced this pull request Jun 17, 2022
Co-authored-by: User123698745 <usr123698745+git@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants