Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use additional headers in the signature payload: #644

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jacobweinstock
Copy link

Use additional headers in the signature payload:

This will allow users to use headers in the signature payload during the verification process. For example, a timestamp header can be used to validate if requests have been sent recently and reduce the risk of message replays.

Fixes: #643

This will allow users to use headers in the signature
payload during the verification process. For example,
a timestamp header can be used to validate if requests
have been sent recently and reduce the risk of message
replays.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
The range body will not execute if the len
of r.Parameter.SignaturePayloadHeaders is 0.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
Cover the Evaluate func with additional headers
for sha1, sha256, sha512.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock marked this pull request as ready for review June 9, 2023 22:27
Dependencies were old.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow additional headers to be part of the signature payload
1 participant