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

Split trust endpoint #82

Open
wants to merge 16 commits into
base: master
from
Open

Conversation

@chaosaffe
Copy link

chaosaffe commented Feb 13, 2020

Intended to resolve the issues with the hard-coded endpoint issues mentioned in #51

PR is heavily based on the work in #63 by @jerrinss5

jerrinss5 and others added 15 commits Apr 27, 2019
…d modified use of access token in the absence of token
…rom www authenticate header
…here anything other 200 or 401 is returned
…ockerconfig file - Portieris will rely on the environment variables for default fallback
…th code
…g the value is invalid
Issue 51:
1. Resolved  using Errorf instead of Fatalf for test cases
2. Made a post call to oauth endpoint first and if that returns a 404 make a get call using the query builder rather than string concatentation
Use the upstream package instead of copying in files
@chaosaffe chaosaffe force-pushed the chaosaffe:split-trust-endpoint branch from e9045dc to 3a1c77c Feb 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.