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

@sonarcloud
Copy link

sonarcloud bot commented Oct 27, 2023

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

Luca Giommi added 29 commits October 27, 2023 15:22
First commit for client iam task
781d985
Added proper deletion of iam clients when deployments fail, and clean…
0c0e0d5 
…ed up the code
Take proper client for orchestrator: it will be from the same issuer …
e7107e6 
…that released the user's token
Added function to check if the idp used is iam
7a91ccf
Added email and uuid in client creation
b455a54
First attemp to clean the iamService class
60b9120
Added the IamClientRequest class and created a Impl file for the IamS…
a413b9c 
…ervice class
Added bean for IamService and cleaned the code with error handling
2c8f24a
Created IamServiceException class and implemented its usage in the code
0803595
Modified and extended the exception handling
de93634
Fixed the association of the client created to the node with the prop…
9c68f32 
…er name
Implemented the possibility to create multiple IAM clients in multipl…
a694b6e 
…e different idp
Covered the case when no iam clients should be created
7bdc7b6
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
Try to create client without passing token to IM
cb9d255
Added function to get client information and to update them
e643a2b
Modified the code to properly write properties on metadata, and modif…
6446434 
…y the template
Addedd assign ownership for clients and use registration_access_token…
cb6e476 
… in http requests
added possibility to set accountId and handled error exception with d…
a8eb69d 
…elete of IAM clients
Start cleaning IMServiceImpl and added useful comments
ffe6bf9
Change accountId into owner and do not kill the deployment if the set…
623e741 
…ting of client owner fails
Clean code with old functions and move IamClientRequest.java and Well…
c250126 
…KnownResponse.java files
Reformatting lines, added deletion of clients when some error occurre…
351916c 
…d and cleaned the code
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
Added new line to restore initial code
72fc4e8
Fix missing parameters in String
e5fec3b
Remove getEndopoint function as it is very similar to the getWellKnow…
d9a309e 
…n function
Integrate Sonar suggestions
879af4c
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.

None yet
2 participants
@lgiommi @maricaantonacci