Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Add HMAC middleware #8

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Add HMAC middleware #8

wants to merge 8 commits into from

Conversation

Muirrum
Copy link
Member

@Muirrum Muirrum commented Jul 13, 2020

Middleware to authenticate requests to the API via HMAC.

Closes #6

@Muirrum Muirrum changed the title Write HMAC middleware Add HMAC middleware Jul 13, 2020
@hkatzdev hkatzdev requested review from hkatzdev and removed request for hkatzdev July 13, 2020 14:36
src/middlewares.rs Outdated Show resolved Hide resolved
src/middlewares.rs Outdated Show resolved Hide resolved
src/middlewares.rs Outdated Show resolved Hide resolved
src/middlewares.rs Show resolved Hide resolved
src/middlewares.rs Outdated Show resolved Hide resolved
src/middlewares.rs Outdated Show resolved Hide resolved
Co-authored-by: Cedric Hutchings <[email protected]>
@Muirrum Muirrum requested a review from cedric-h July 13, 2020 19:49
Copy link
Contributor

@cedric-h cedric-h left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, I just think we need to be consistent about how we do multiple imports from the same source.

src/middlewares.rs Outdated Show resolved Hide resolved
Muirrum and others added 2 commits July 13, 2020 17:44
Co-authored-by: Cedric Hutchings <[email protected]>
In my suggestion, I failed to specify that a line was taking the place of three separate lines, not just the one highlighted.
@Muirrum Muirrum marked this pull request as draft July 22, 2020 13:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Authentication
2 participants