-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
If HTTP signature is wrong and webfinger cache is stale, retry with resolve #5129
If HTTP signature is wrong and webfinger cache is stale, retry with resolve #5129
Conversation
…esolve If the signature could not be verified and the webfinger of the account was last retrieved longer than the cache period, try re-resolving the account and then attempting to verify the signature again
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.
While it doesn't look wrong to me, I fail to see what it solves. Indeed, account_from_key_id
already updates remote data (provided old enough last_webfingered_at
) by calling ResolveRemoteAccount
.
On a side note, I have noticed the following files include SignatureVerification
without seemingly using it:
app/controllers/accounts_controller.rb
app/controllers/stream_entries_controller.rb
@ThibG Uh no, account_from_key_id only calls ResolveRemoteAccountService for OStatus keyId format which is not used by Mastodon anymore. |
…esolve (mastodon#5129) If the signature could not be verified and the webfinger of the account was last retrieved longer than the cache period, try re-resolving the account and then attempting to verify the signature again
…esolve (mastodon#5129) If the signature could not be verified and the webfinger of the account was last retrieved longer than the cache period, try re-resolving the account and then attempting to verify the signature again
…esolve (mastodon#5129) If the signature could not be verified and the webfinger of the account was last retrieved longer than the cache period, try re-resolving the account and then attempting to verify the signature again
…esolve (mastodon#5129) If the signature could not be verified and the webfinger of the account was last retrieved longer than the cache period, try re-resolving the account and then attempting to verify the signature again
If the signature could not be verified and the webfinger of the account was last retrieved longer than the cache period, try re-resolving the account and then attempting to verify the signature again