Skip to content
This repository has been archived by the owner on Jan 24, 2018. It is now read-only.

Add Auth0 docs and ~b64decoding #1620

Merged
merged 5 commits into from
Mar 11, 2017
Merged

Add Auth0 docs and ~b64decoding #1620

merged 5 commits into from
Mar 11, 2017

Conversation

david4096
Copy link
Member

@david4096 david4096 commented Mar 10, 2017

Thank you SOOO much @Welliton309 for writing these docs. The community could really benefit from easier authentication/authorization solutions!

This PR fixes a problem found in #1565 that Auth0 changed the formatting of their client secrets (no longer b64 encoded). This PR fixes that to support all future Auth0 clients.

The links to the images are hosted by github's cache, which is probably not a great solution and they should be checked in.

TODO

Move images to static directory.

@david4096 david4096 changed the title 1565 auth0 Add Auth0 docs and ~b64decoding Mar 10, 2017
@david4096 david4096 force-pushed the 1565_auth0 branch 2 times, most recently from e253983 to e4cf993 Compare March 10, 2017 21:32
With external links to images
Close ga4gh#1565
@codecov-io
Copy link

Codecov Report

Merging #1620 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1620      +/-   ##
==========================================
- Coverage   86.65%   86.65%   -0.01%     
==========================================
  Files          33       33              
  Lines        7471     7469       -2     
  Branches      926      926              
==========================================
- Hits         6474     6472       -2     
  Misses        805      805              
  Partials      192      192
Impacted Files Coverage Δ
ga4gh/server/auth/init.py 66.66% <ø> (-0.63%)
ga4gh/server/serverconfig.py 100% <100%> (ø)

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 ec57524...44c3e82. Read the comment docs.

@kozbo
Copy link
Contributor

kozbo commented Mar 11, 2017

+1, and merging

@kozbo kozbo merged commit ad806f7 into ga4gh:master Mar 11, 2017
@david4096 david4096 deleted the 1565_auth0 branch March 11, 2017 00:53
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.

3 participants