- Notifications
You must be signed in to change notification settings - Fork928
chore: support multi-org group sync with runtime configuration#14578
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes from1 commit
Commits
Show all changes
38 commits Select commitHold shift + click to select a range
99c97c2
wip
Emyrkbfddeb6
begin group sync main work
Emyrkf2857c6
initial implementation of group sync
Emyrk791a059
work on moving to the manager
Emyrk4326e9d
fixup compile issues
Emyrk6d3ed2e
fixup some tests
Emyrk0803619
handle allow list
Emyrk596e7b4
WIP unit test for group sync
Emyrkb9476ac
fixup tests, account for existing groups
Emyrkee8e4e4
fix compile issues
Emyrkd5ff0f7
add comment for test helper
Emyrk86c0f6f
handle legacy params
Emyrk2f03e18
make gen
Emyrkec8092d
cleanup
Emyrkd63727d
add unit test for legacy behavior
Emyrk2a1769c
work on batching removal by name or id
Emyrk640e86e
group sync adjustments
Emyrkc544a29
test legacy params
Emyrk476be45
add unit test for ApplyGroupDifference
Emyrk164aeac
chore: remove old group sync code
Emyrk986498d
switch oidc test config to deployment values
Emyrk290cfa5
fix err name
Emyrkc563b10
some linting cleanup
Emyrkd2c247f
dbauthz test for new query
Emyrk12685bd
fixup comments
Emyrkbf0d4ed
fixup compile issues from rebase
Emyrkf95128e
add test for disabled sync
Emyrk88b0ad9
linting
Emyrk6491f6a
chore: handle db conflicts gracefully
Emyrkbd23288
test expected group equality
Emyrka390ec4
cleanup comments
Emyrka0a1c53
spelling mistake
Emyrka86ba83
linting:
Emyrk0df7f28
add interface method to allow api crud
Emyrk7a802a9
Remove testable example
Emyrk611f1e3
fix formatting of sql, add a comment
Emyrk7f28a53
remove function only used in 1 place
Emyrk41994d2
make fmt
EmyrkFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
add interface method to allow api crud
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commit0df7f28209e5b32747f7859871cfa7c954faf6f6
There are no files selected for viewing
3 changes: 3 additions & 0 deletionscoderd/idpsync/group.go
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
5 changes: 4 additions & 1 deletioncoderd/idpsync/idpsync.go
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
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.