Optional parameter for HTTP Signatures if key changes #15
Labels
No labels
Critical Bug
Documentation
Feature
Small bug
tests
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
babka/activitycolander#15
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
There are two possibilities for an actor's public key changing.
The first is that there's been a change by the server. The other is that there's been an attack and a malicious party has decided to take over.
An example of the second case would be if there was DNS poisoning, redirecting clients (such as ActivityColander) to a malicious site where the new public key is.
Therefore, there should be an optional parameter to the check that does not do a key lookup if the check fails, eg
HTTP_SIGNATURE_VERIFICATION_FETCHshould could be a binary value, 1 for true (default) and 0 for false.@CSDUMMI pls update this issue
The HttpSignatureCheck now has a
no_refetch = falseoption.If a signature verification failed and this option is false, a fetch of the public key will take place.