-
Notifications
You must be signed in to change notification settings - Fork 50
[Feature] Implement U2M Authentication in the Go SDK #1108
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
Merged
+1,853
−286
Merged
Changes from all commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
4a538e3
work
mgyucht 70ef496
Merge branch 'main' into move-u2m-to-sdks
mgyucht 3550329
fix tests
mgyucht 05bd576
Merge branch 'main' into move-u2m-to-sdks
mgyucht 0688c3b
fix test
mgyucht 35b8365
respond to some changes
mgyucht 47f2fb2
more work
mgyucht e883616
work
mgyucht 181f88b
basic tests
mgyucht 74592c3
some tests
mgyucht 9b5913c
one more test case
mgyucht 73c73af
more tests
mgyucht 63bf521
comment
mgyucht 2622989
mutex
mgyucht 359a5d0
documentation
mgyucht 44af89f
docs
mgyucht bd7303e
fix test names
mgyucht 323fb61
work
mgyucht 3905c5d
fix tests
mgyucht f53fb84
comments and test fixes
mgyucht ef8c3f3
fix
mgyucht 0ff447d
fix
mgyucht 6c2ed96
work
mgyucht e9f3732
better error message
mgyucht 19a34ce
fix test
mgyucht c7b5155
fix
mgyucht 37b44b1
tweak
mgyucht 36ea3dc
work
mgyucht fc87393
fix test
mgyucht 945151f
undo move
mgyucht 356a7c9
remove extra context
mgyucht 7441bd1
work
mgyucht 3550bba
address comment
mgyucht fcb031f
fix
mgyucht 83e4141
more tweaks
mgyucht 51a5b08
work
mgyucht 62acd68
simpler
mgyucht f506510
tweaks
mgyucht d70dac4
fix test
mgyucht 373daf3
Merge branch 'main' into move-u2m-to-sdks
mgyucht 2f0ebbb
work
mgyucht 21df9ec
Merge branch 'main' into move-u2m-to-sdks
mgyucht 6ff1f3d
fmt
mgyucht f08c2b9
work
mgyucht de60fcf
fix
mgyucht 11380f0
remove
mgyucht 44c2ca7
fix
mgyucht 45b7e55
changelog
mgyucht 51da35a
Merge branch 'main' into move-u2m-to-sdks
mgyucht e37d457
clean up
mgyucht 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 was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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
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.
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.
This is moved to credentials/oauth/oidc.go: https://github.com/databricks/databricks-sdk-go/pull/1108/files#diff-dbbf458b1ab6c594fd438f9281715b863447375ede0366e94a7159052c240defR1. It has been separated into two methods.