-
Notifications
You must be signed in to change notification settings - Fork 305
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
Added user attributes, client auth resources, service account roles #104
Merged
mrparkers
merged 54 commits into
mrparkers:master
from
AndrewChubatiuk:client-resources
May 14, 2019
Merged
Changes from 30 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
3dd442f
initial commit
6909b20
go fmt
853524d
fixed resource body params
5555d95
fixed resource id
bcbd78b
upgraded keycloak and added example
81d5740
minor fix
fcce373
fixed resources
1e41445
fixed web origins
2c1c57b
added openid client authorization resources
78f84e9
resolved merge conflicts
a96715e
added examples
50d0dd4
Merge branch 'client-resources' of github.com:AndrewChubatiuk/terrafo…
ae21d62
updated endpoints
2ca6644
minor fix
6ca699f
fixed examples
558218b
removed unused util function
8edfe09
minor fix
ea1eae2
enable authorization service
1b25743
minor fix
cb87711
fixed tests for keycloak 6
5b540dd
minor fix
9131e7f
fixed tests
fdfe893
minor fix
ed8a631
minor fix
62d0e1b
minor fix
49697d0
updated service account role logic
3f5ec97
added resource_server_id
4c1606c
updated examples
c5f7396
minor fix
1a1a724
resolved conflicts
0e38f7c
Merge branch 'master' of github.com:AndrewChubatiuk/terraform-provide…
ff257fe
Merge branch 'master' into client-resources
AndrewChubatiuk cdcd08a
renamed new providers, PR comments fixes
e3c4562
Merge branch 'client-resources' of github.com:AndrewChubatiuk/terrafo…
eb3ea2a
fmt fix
484e7c2
pr comments
0084dd8
added tests
13fa442
minor fix
51edd05
minor fix
5f7328c
minor test fix
a1fd8bc
fixed getclientbyName
3cd9b3d
fixed delete request
288a4a7
Merge branch 'client-resources' of github.com:AndrewChubatiuk/terrafo…
adf1a3b
fixed auth policy test
eda5f46
test
26567d3
fixed service account tests
69a5c4f
fixed typo
0cdf594
minor fix
c9ef71b
minor update
517452b
minor fix
fae15f0
fixed delete request for service account role
d7c0e21
minor fix
8919a4e
trigger build
a688cc0
added default resource removal
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 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
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
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
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.
Since you added the
params
argument to theget
method, would you mind updating this line to use that argument for thesearch
param here?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.
Done