Skip to content
This repository was archived by the owner on Jul 16, 2021. It is now read-only.

Adds JWT Auth mechanmism using GitHub Oauth#361

Merged
prydonius merged 4 commits intohelm:masterfrom
prydonius:auth
Sep 26, 2017
Merged

Adds JWT Auth mechanmism using GitHub Oauth#361
prydonius merged 4 commits intohelm:masterfrom
prydonius:auth

Conversation

@prydonius
Copy link
Member

@prydonius prydonius commented Sep 24, 2017

Protects API methods for:

  • adding and delete repos
  • viewing, creating and deleting Helm releases

The JWT is stored in an httpOnly cookie and is only valid for 2 hours. A second cookie, accessible via JS, is used in the UI to determine if the user is logged in or not.

@codecov-io
Copy link

codecov-io commented Sep 24, 2017

Codecov Report

Merging #361 into master will decrease coverage by 0.03%.
The diff coverage is 86.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #361      +/-   ##
==========================================
- Coverage   87.67%   87.64%   -0.04%     
==========================================
  Files          18       19       +1     
  Lines         844      874      +30     
==========================================
+ Hits          740      766      +26     
- Misses         68       70       +2     
- Partials       36       38       +2
Impacted Files Coverage Δ
src/api/middleware/authgate.go 86.66% <86.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7c4045...df9967d. Read the comment docs.

Adnan Abdulhussein added 2 commits September 25, 2017 16:47
- load secrets from env
- update dev env to support ignored .env file
- add auth/verify and auth/logout to manage auth state
@prydonius prydonius merged commit b46f308 into helm:master Sep 26, 2017
@prydonius prydonius deleted the auth branch September 26, 2017 09:13
prydonius pushed a commit that referenced this pull request Dec 6, 2018
prydonius pushed a commit that referenced this pull request Dec 6, 2018
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.

2 participants