Fix for another round of reddit API fuckery#7
Open
dziban303 wants to merge 3 commits into
Open
Conversation
Change URL to use old.reddit.com Change date format to use ISO 8601 yyyy-MM-dd format
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey Victor, been awhile. I noticed Reddit decided to play around with their API security and now randos and their scripts are seemingly out of luck if they want to access reddit assets unauthenticated. I tried just adding useragent stuff at first, but Cloudflare or whoever is too smart for me.
So I rewrote some portions of the script to make use of oauth2. Users will have to create an app at the reddit dev console and input the client ID and secret, and their username and password, as part of the initial
-configflow. Unfortunately these variables are stored in plaintext in the config file because I couldn't figure out a more secure solution, and honestly mate I felt like further effort was a waste of time: I don't think very many people are using this script, lol.Oh, I also added
-showhelpbecause it apparently was never implemented.Anyway it's working now, here's the PR to bring upstream up to speed. I know you said you don't use Windows anymore but if you do, some further testing wouldn't hurt!
Cheers mate
P.S., AI disclosure, claude helped with the oauth stuff, but I did carefully go over the changes it made.