-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add initial version of login to Octopus with OIDC or API keys #3
Merged
Merged
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
02604f2
Wip login to octopus
geofflamrock 5657a20
Add pr trigger to get workflow to show up
geofflamrock e9bd07f
Remove PR trigger
geofflamrock bca99fe
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock 55e4ebe
Change to info logging
geofflamrock 4c2b79f
Add id-token permissions
geofflamrock 59d4d71
Fix permissions
geofflamrock 797fa3d
Add content type header
geofflamrock 1c3fd78
Attempting to see the error
geofflamrock 3d4e189
Fix request body formatting
geofflamrock e919dbb
Add debug logging
geofflamrock 993cc60
Make debug messages info
geofflamrock 64f6039
Remove debugging message
geofflamrock a7e3c58
Add output variables and print env vars
geofflamrock d63ab39
Better error handling and refactoring to include test context
geofflamrock 63f3fbd
More testing of action
geofflamrock bf168b8
Fix missing space parameter
geofflamrock 78eec29
Use defaults from repo if not supplied to make testing easier
geofflamrock 525a8b8
Improves eslint configuration and cleanup
geofflamrock 4222799
wip adding tests
geofflamrock 9942b77
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock 9d0f8b0
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock 66027ba
Fix build
geofflamrock 230e394
Add changeset
geofflamrock 7112c7b
More tests
geofflamrock 363360b
Adds login details to readme
geofflamrock 000c850
Fixes examples
geofflamrock f55c0f3
Adds linting to ci
geofflamrock 80b4d08
Add test report
geofflamrock 1e1da5d
Fix up ci test reporting
geofflamrock 0b80bbb
Set correct permissions
geofflamrock e254244
Change to different test report action
geofflamrock 79d9af8
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock 5b57472
Update lockfile
geofflamrock d5dc109
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock 183debb
Merge from main
geofflamrock d049e90
Use openid configuration
geofflamrock 4b9c023
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock 2b4f2f0
Add user agent
geofflamrock c942b65
Merge branch 'geoffl/use-openid-configuration' into geoffl/add-login
geofflamrock 440bcd1
Add user agent to openid config request
geofflamrock ffa3b2a
Fix casing
geofflamrock b48e7f1
Merge remote-tracking branch 'origin/main' into geoffl/add-login
geofflamrock File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains 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 |
---|---|---|
@@ -0,0 +1,5 @@ | ||
--- | ||
"@octopusdeploy/login": minor | ||
--- | ||
|
||
Add initial version of login to Octopus with OpenID Connect or API Keys |
This file contains 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 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 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 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 |
---|---|---|
|
@@ -127,3 +127,6 @@ out | |
.yarn/build-state.yml | ||
.yarn/install-state.gz | ||
.pnp.* | ||
|
||
# Test outputs | ||
test-results/ |
This file contains 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 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have adjusted the eslint config to match up more closely to what we use elsewhere in the Octopus portal. It's not 100% the same, but much closer.