-
Notifications
You must be signed in to change notification settings - Fork 48
Add authentication and SSL/TLS to ServiceControl #5197
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
Open
warwickschroeder
wants to merge
42
commits into
master
Choose a base branch
from
genxp-3600-add-authentication
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+12,312
−87
Open
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
2b2d96d
Add initial authentication supporting JWT tokens, OpenID Connect, OAu…
warwickschroeder 23703ea
Add ServicePulse-specific OIDC configuration and endpoint
jasontaylordev 7fb96c5
Update src/ServiceControl/App.config
jasontaylordev b0863c5
Update src/ServiceControl/App.config
jasontaylordev c6b4ded
Remove SP enabled flag
jasontaylordev 26085fe
Update approved routes and settings
jasontaylordev 2c185f4
Allow multiple api scopes, or none. Add client audience config setting
warwickschroeder af70ae9
Add auth to other instances
jasontaylordev a1298ad
Rename to ApiScopes
jasontaylordev 2d7fbd7
Add additional options for flexible and secure hosting; SSL/TLS, Reve…
warwickschroeder 2d2f6a7
Remove previously added rate limit for anon api
warwickschroeder 56dba7c
Forward auth header
jasontaylordev 1fc05e7
Allow Anon for CheckRemotes
jasontaylordev 10afc1b
Remove unused rate limiting middleware
jasontaylordev 4fdc321
MapControllers correctly
jasontaylordev 1122bfe
Upgrade package
jasontaylordev 182f38b
Update local testing files. Add debug endpoint for dev.
warwickschroeder e46aeba
Update reverse proxy test file
warwickschroeder c98ddbd
Update HTTPS config and documentation
warwickschroeder 1b3e851
Update documentation for authentication
warwickschroeder fb75ae7
Add forward header tests for all instances. Add links to additional d…
warwickschroeder 34ea48c
Add more manual testing scenarios to docs. Rename files.
warwickschroeder f534d93
Clean doc formatting. Update hosting guide.
warwickschroeder 49056a0
Update internal auth docs. Fix issue with server-to-server remote ins…
warwickschroeder 47451b7
Move public docs content to the docs PR
warwickschroeder 39e7fff
Clean servicecontrol hosting guide. Move content to docs RP (WIP)
warwickschroeder e88f23b
Move public docs into docs PR
warwickschroeder bc9255d
Clean forward header tests and add comments
warwick-exia 72d2122
Merge branch 'master' into genxp-3600-add-authentication
warwickschroeder 4c18d69
Fix build issues
warwick-exia 98e79f6
fix breaking tests. Add unit tests for security settings
warwick-exia 1b65474
Add security acceptance tests for Primary instance
warwick-exia 2c69830
Add CORS acceptance tests to audit and monitoring.
warwickschroeder 9415d8e
Add HTTPS acceptance tests to audit and monitoring
warwickschroeder fc4ed1f
Add OpenID Conntect acceptance tests to audit and monitoring
warwickschroeder 001fd35
Move some documentation to public docs
warwickschroeder 2376f45
Small tweaks off review
warwickschroeder 231e87a
Add comments. Update security setting validation.
warwickschroeder 45aa183
Update tests off of review
warwickschroeder 27ce810
Fix API approval tests
warwickschroeder c2fecfb
Update from feedback
warwickschroeder 61f7da6
Updates from testing. Add additional logging to support testing. Upda…
warwickschroeder File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.