-
Notifications
You must be signed in to change notification settings - Fork 26
Add authentication and SSL/TLS to ServicePulse #2729
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
30
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.
+9,698
−65
Open
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
13e5b1b
Replace moment with dayjs
johnsimons 9ec501e
WIP - Add authentication
warwickschroeder 919d422
Load OIDC configuration from ServiceControl API instead of local env
jasontaylordev b9d5fff
Adding support for enabling/disabling authentication
jasontaylordev 9f43613
Add lint rule to check for raw fetch calls.
warwickschroeder 9a1cf63
Add user profile menu item. Fix timing issue when SC isnt available. …
warwickschroeder 607f692
Allow multiple api scopes. Add audience config which is needed for so…
warwickschroeder 9754f0f
Package updates
jasontaylordev c92d312
Merging
jasontaylordev ed1cf35
Add HTTPS config with documentation
warwickschroeder 8b4b01e
Update auth and https documentation
warwickschroeder b4cfcc9
Update doc naming
warwickschroeder fbabd1f
Fix doc namie
warwickschroeder 279b0a7
Update auth testing docs. Add anon logout page.
warwickschroeder d351ecd
Add authentication tests
warwickschroeder 06f8591
Update authentication-testing.md
jasontaylordev 1dcaca6
Fix user profile dropdown appearing behind dashboard items
jasontaylordev 29e1db4
Doc update
jasontaylordev d8e968d
Update package-lock.json
jasontaylordev 0d07fe8
Move public doco to docs PR
warwickschroeder 477556a
Update test docs. Split up auth tests to individual scenario tests.
warwickschroeder 2de747a
Fix build error
warwickschroeder 6bcd0e1
Add unit and acceptance tests for ServicePulse .net framework and .ne…
warwickschroeder d5b9b0e
Update vulnerable package
warwickschroeder a441e74
Update dependancies
warwickschroeder b081856
Fix build issues. Warnings upgraded to error.
warwickschroeder 291bdd4
Update ci to check for .net8 test projects (up from .net7)
warwickschroeder ecddb4a
Update tests from review. Add some comments
warwickschroeder 6f14cc3
Clean frontend tests. Fix build issues
warwickschroeder 7b76e86
Update from feedback. Add setting logging. Upgrade SC urls to https i…
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -101,3 +101,10 @@ src/scaffolding.config | |
|
|
||
| # Visual Studio Code | ||
| .vscode | ||
|
|
||
| # AI | ||
| .claude | ||
| CLAUDE.md | ||
|
|
||
| # local testing | ||
| .local | ||
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.