Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Client iam #441

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open

Client iam #441

wants to merge 44 commits into from

Conversation

lgiommi
Copy link

@lgiommi lgiommi commented Oct 25, 2023

Implemented the feature of creating IAM clients by the orchestrator

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

lgiommi added 29 commits October 27, 2023 15:22
@lgiommi
First commit for client iam task
781d985
@lgiommi
Added proper deletion of iam clients when deployments fail, and clean…
0c0e0d5 
…ed up the code
@lgiommi
Take proper client for orchestrator: it will be from the same issuer …
e7107e6 
…that released the user's token
@lgiommi
Added function to check if the idp used is iam
7a91ccf
@lgiommi
Added email and uuid in client creation
b455a54
@lgiommi
First attemp to clean the iamService class
60b9120
@lgiommi
Added the IamClientRequest class and created a Impl file for the IamS…
a413b9c 
…ervice class
@lgiommi
Added bean for IamService and cleaned the code with error handling
2c8f24a
@lgiommi
Created IamServiceException class and implemented its usage in the code
0803595
@lgiommi
Modified and extended the exception handling
de93634
@lgiommi
Fixed the association of the client created to the node with the prop…
9c68f32 
…er name
@lgiommi
Implemented the possibility to create multiple IAM clients in multipl…
a694b6e 
…e different idp
@lgiommi
Covered the case when no iam clients should be created
7bdc7b6
@lgiommi
Added the possibility to set scopes and refactored the code to get th…
8caa0d0 
…e endpoint urls from the json obtained contacting the wellknown endpoint
@lgiommi
Try to create client without passing token to IM
cb9d255
@lgiommi
Added function to get client information and to update them
e643a2b
@lgiommi
Modified the code to properly write properties on metadata, and modif…
6446434 
…y the template
@lgiommi
Addedd assign ownership for clients and use registration_access_token…
cb6e476 
… in http requests
@lgiommi
added possibility to set accountId and handled error exception with d…
a8eb69d 
…elete of IAM clients
@lgiommi
Start cleaning IMServiceImpl and added useful comments
ffe6bf9
@lgiommi
Change accountId into owner and do not kill the deployment if the set…
623e741 
…ting of client owner fails
@lgiommi
Clean code with old functions and move IamClientRequest.java and Well…
c250126 
…KnownResponse.java files
@lgiommi
Reformatting lines, added deletion of clients when some error occurre…
351916c 
…d and cleaned the code
@lgiommi
Add proper log messages. Do not kill the deployment if there are no o…
1fd2793 
…rchestrator clients related to the issuer requested by the user
@lgiommi
Added new line to restore initial code
72fc4e8
@lgiommi
Fix missing parameters in String
e5fec3b
@lgiommi
Remove getEndopoint function as it is very similar to the getWellKnow…
d9a309e 
…n function
@lgiommi
Integrate Sonar suggestions
879af4c
@lgiommi
Define static strings to make the code more readable
1490b05
@lgiommi lgiommi marked this pull request as ready for review October 27, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lgiommi @maricaantonacci