-
Notifications
You must be signed in to change notification settings - Fork 0
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
[SELC-4590] feat: Added V2 API for tokenExchange and billing tokenExchange #390
Conversation
Quality Gate failedFailed conditions |
# Conflicts: # connector/rest/src/main/java/it/pagopa/selfcare/dashboard/connector/rest/UserConnectorImpl.java
} | ||
} | ||
|
||
private Institution retrieveInstitution(String institutionId, List<ProductGrantedAuthority> productGrantedAuthorities, boolean isBillingToken) { |
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.
Could we move this logic into a separate service (adding also a mapper) avoding to have private methods - or at least a part of them - into this service?
|
||
@GetMapping(value = "exchange") | ||
@ResponseStatus(HttpStatus.OK) | ||
@ApiOperation(value = "", notes = "${swagger.dashboard.token.api.exchange}") |
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.
As mentioned in previous pull requests, we should add a specific operation identifier for the endpoints
Quality Gate failedFailed conditions |
List of Changes
Added V2 API for tokenExchange and billing tokenExchange
Motivation and Context
With a view to migrating to API v2, it is necessary to expose this api as internally it uses user authorities to perform business logic
How Has This Been Tested?
local env.
Screenshots (if appropriate):
Checklist: