Skip to content
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

Is there a way for server to enforce the headers? #1

Open
yanfali opened this issue Aug 13, 2021 · 0 comments
Open

Is there a way for server to enforce the headers? #1

yanfali opened this issue Aug 13, 2021 · 0 comments

Comments

@yanfali
Copy link

yanfali commented Aug 13, 2021

Thank you for this plugin. I've been looking over the code and wondering if there's a way to specify a minimum list of headers that the server can require in the plugin. If I'm reading the code correctly, it appears the client can choose which headers are used to calculate the HMAC and the server won't enforce any minimum set of headers while parsing the Auth string. Did I misunderstand that?

My concern is this could lead to a downgrade attack, where an incorrectly configured client could just specify fewer fields, and the server would happily honor the request. I assume specifying more fields means you potentially have more entropy and the hmac is more resistant to attacks?

Also if I understand the spec correctly, one would need to issue an OPTIONS request, much like CORS, to get a list of the headers needed for the signature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant