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

Add authentication framework #44

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

Add authentication framework #44

wants to merge 2 commits into from

Conversation

jeremyrsmith
Copy link
Collaborator

  • Add the ability to place filters before the HTTP service, configurable
    at the request level
  • Add the concept of an Authorizer, which is just a filter.
  • Add some initial Authorizers - Bearer (for pre-shared Bearer tokens)
    and MAC (for pre-shared MAC tokens)

A future PR should contain a full OAuth2 flow implemented in an
Authorizer.

- Add the ability to place filters before the HTTP service, configurable
  at the request level
- Add the concept of an Authorizer, which is just a filter.
- Add some initial Authorizers - Bearer (for pre-shared Bearer tokens)
  and MAC (for pre-shared MAC tokens)

A future PR should contain a full OAuth2 flow implemented in an
Authorizer.
@codecov-io
Copy link

codecov-io commented Oct 4, 2016

Current coverage is 54.48% (diff: 20.23%)

Merging #44 into master will decrease coverage by 8.87%

@@             master        #44   diff @@
==========================================
  Files             9         10     +1   
  Lines           262        312    +50   
  Methods         259        308    +49   
  Messages          0          0          
  Branches          3          4     +1   
==========================================
+ Hits            166        170     +4   
- Misses           96        142    +46   
  Partials          0          0          

Powered by Codecov. Last update bdecef0...e747ec5

@ilya-murzinov
Copy link
Contributor

@jeremyrsmith is it possible to write a test for Bearer and MAC authorizers?

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

Successfully merging this pull request may close these issues.

3 participants