Update auth.nim #1164
Merged
Update auth.nim #1164
Conversation
This file contains 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
KTachibanaM
added a commit
to sekai-soft/nitter
that referenced
this pull request
Feb 7, 2024
Avoid expiring the tokens for now. See: - zedeus#983 (comment) - zedeus#1155 (comment) Thanks @cmj
|
Might as well. |
|
Is this going to get merged into master at some point? This commit is what's packaged in Nixpkgs, I presume because it fixes something important? |
zedeus
added a commit
that referenced
this pull request
Feb 5, 2025
* Replace tokens with guest accounts, swap endpoints * Filter out account limits that already reset * Track rate limits, reset after 24 hours * Add missing limitedAt assignment * Add guest accounts variable to GitHub action * Unify some guest account logs * Fix compilation error * Reduce max concurrent pending requests per account * Fix compilation on older versions of Nim * Tweak /.tokens, add amount of limited accounts * Switch back to old user search endpoint * Fix tweetDetail stats * Switch to using typeahead for user search * Revert "Switch to using typeahead for user search" This reverts commit a3e11e3. * Switch to TweetDetail for tweets * Cleanup * Fix cards not being displayed * Improve .tokens output * Handle ProtocolError and BadClientError equally * Improve rate limit handling, minor refactor * Fix accounts logging * Add 429 logging * Add empty oauth token logging * Fix negative pending requests bug * Add media proxying error logging * Shorten media proxy error log * Retry rate limited requests with different account * Fix crash on missing quote tweet data crash * Increase photo rail cache ttl * Improve guest accounts loading, add JSONL support * Improve Liberapay css for Firefox compatibility * fix(nitter): add graphql user search (#1047) * fix(nitter): add graphql user search * fix(nitter): rm gitignore 2nd guest_accounts * fix(nitter): keep query from user search in result. remove personal mods * fix(nitter): removce useless line gitignore * Fix multi-user timelines * Update nimcrypto and jsony * Bump minimum Nim version * Replace /.tokens with /.health and /.accounts * Fix compilation on Nim 1.6.x * Run tests on multiple Nim versions * Fix guest accounts CI setup * Fix guest accounts CI setup attempt 2 * Prevent annoying warnings on devel * Fix missing CI file argument * Fix guest accounts CI setup attempt 3 * Remove redundant "active" field from /.health * Fix guest accounts CI setup attempt 4 * Fix outdated test * Move CI workflow to buildjet * Purge expired accounts after parsing * Don't requests made before reset * Fix #670 * Bump Nim in the ARM64 Dockerfile, add nitter user * Fix ARM64 Dockerfile versions * Fix #1106 Closes #831 * Enable disabled tests * Add simple job_details card support * Fix jobDetails error on old Nim versions * Add support for business and gov verification Also improve icon rendering on Firefox * Fix compilation with old Nim again * Fix search queries in user search RSS feeds (#1126) Fixes #992 * Fix broken video playback by forcing fmp4 * Fix for #1147, Proxy for audio URL and upgrade hls.js (#1178) * Revert "Fix broken video playback by forcing fmp4" This reverts commit 52db03b. * Fix audio url in video m3u8 * Upgrade hls.js to 1.5.1 and use full version * Update auth.nim (#1164) Avoid expiring the tokens for now. See: - #983 (comment) - #1155 (comment) Thanks @cmj * Replace old v1 photo rail API with gql * Raise NoAccountsError when all accounts limited * Move limited flag to be account-level * Capture "account locked" API error * Update authority header * Add regex for x.com links * Update endpoint versions, switch tweet endpoint * Return 403 on hmac error * Add lazy loading for images * Fix missing video thumbnail being too small * Silence 404 proxy errors * Skip null fetch errors * Increase max concurrent reqs per account * Rename accounts/guest accounts to sessions The new file loaded by default is now ./sessions.jsonl JSONL is also required, .json support dropped. * Update Dockerfiles * Update README, added an important note * Add Python script to create account sessions * Change main page search to "Tweets" search * Remove old tokenCount from nitter.example.conf * Fix tests * Fix empty image URLs in photo rail --------- Co-authored-by: LS <[email protected]> Co-authored-by: blankie <[email protected]> Co-authored-by: jackyzy823 <[email protected]> Co-authored-by: somini <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Avoid expiring the tokens for now.
See:
Thanks @cmj
See #1155 #983