-
Notifications
You must be signed in to change notification settings - Fork 232
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
feat: add STS Accounts API #4493
feat: add STS Accounts API #4493
Conversation
* @param parameters Optional generator arguments, such as a salt value | ||
* @return a randomly generated client secret. | ||
*/ | ||
String generateClientSecret(@Nullable Object parameters); |
Check notice
Code scanning / CodeQL
Useless parameter Note
.../main/java/org/eclipse/edc/iam/identitytrust/sts/spi/service/StsTokenGenerationProvider.java
Fixed
Show fixed
Hide fixed
.../org/eclipse/edc/api/iam/identitytrust/sts/accounts/controller/StsAccountsApiController.java
Outdated
Show resolved
Hide resolved
...y-trust-sts/identity-trust-sts-accounts-api/src/main/resources/sts-accounts-api-version.json
Outdated
Show resolved
Hide resolved
...y-trust-sts/identity-trust-sts-accounts-api/src/main/resources/sts-accounts-api-version.json
Show resolved
Hide resolved
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.
just minor nits
...ccounts-api/src/main/resources/META-INF/services/org.eclipse.edc.spi.system.ServiceExtension
Outdated
Show resolved
Hide resolved
...ql/src/main/java/org/eclipse/edc/iam/identitytrust/sts/store/SqlStsClientStoreExtension.java
Outdated
Show resolved
Hide resolved
.../main/java/org/eclipse/edc/iam/identitytrust/sts/spi/service/StsTokenGenerationProvider.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Enrico Risa <enrico.risa@gmail.com>
.../main/java/org/eclipse/edc/iam/identitytrust/sts/spi/service/StsTokenGenerationProvider.java
Fixed
Show fixed
Hide fixed
* @return The {@link TokenGenerationService} | ||
*/ | ||
TokenGenerationService tokenGeneratorFor(StsClient client); | ||
TokenGenerationService tokenGeneratorFor(StsAccount stsAccount); |
Check notice
Code scanning / CodeQL
Useless parameter Note
What this PR changes/adds
adds the STS Accounts API (+controller) to manage STS Accounts (formerly known as StsClients). To achieve that, the
StsAccountService
was extended to accommodate the new functionality like updating, deleting etc.Why it does that
a way to manage STS Accounts is required
Further notes
StsClient...
->StsAccount...
AuthenticationService
for the"sts-accounts-api"
contextLinked Issue(s)
Closes # <-- insert Issue number if one exists
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.