adds support azure multi-tenancy
Type of change
-
Feature
Does this introduce a change in the core logic?
- [YES]
Does this introduce a change in the cloud provider implementation, if so which cloud?
-
Azure
Does this introduce a breaking change?
- [NO]
Any other useful information
some refactoring in core due to the transition to the latest version os-core-common
Edited by Aliaksei Darafeyeu
Merge request reports
Activity
- Resolved by Aliaksei Darafeyeu
- Resolved by Aliaksei Darafeyeu
added 4 commits
-
958e67dc...e352f119 - 2 commits from branch
master
- e8af485e - adds support azure multi-tenancy
- b61ac43e - updates libs version
-
958e67dc...e352f119 - 2 commits from branch
added 1 commit
- d4a51be0 - replaces CredentialHeadersProvider with NotificationFilter
added 21 commits
-
d4a51be0...3a37042e - 18 commits from branch
master
- 806213f8 - adds support azure multi-tenancy
- 99ef008e - updates libs version
- 428c9dc7 - replaces CredentialHeadersProvider with NotificationFilter
Toggle commit list-
d4a51be0...3a37042e - 18 commits from branch
added 8 commits
-
e19e15c0...3dc08349 - 6 commits from branch
master
- 03cc50e5 - adds support azure multi-tenancy
- fba86914 - fix looping error for handshake RQ
-
e19e15c0...3dc08349 - 6 commits from branch
@komakkar @harshit283 could you please have a look ?
@komakkar @harshit283 reminder
Edited by Aliaksei Darafeyeu
59 60 String userId = getUserId(); 61 if (userId != null) { 62 contextMap.put("user-id", userId); 63 } 64 65 return contextMap; 66 } 67 68 /** 69 * Get user ID from Authorization payload (JWT token). 70 * 71 * @return the user ID 72 */ 73 private String getUserId() { 74 JWTClaimsSet claimsSet = authUtils.getClaimsFromJwtToken(dpsHeaders.getAuthorization()); added 4 commits
-
fba86914...c7cff651 - 2 commits from branch
master
- 1e558205 - adds support azure multi-tenancy
- 80dca06f - fix looping error for handshake RQ
-
fba86914...c7cff651 - 2 commits from branch
28 29 public class EntitlementsClientFactory extends AbstractFactoryBean<IEntitlementsFactory> { 29 30 30 31 @Autowired 31 private IAppProperties config; 32 private IAppProperties config; 33 @Autowired 34 private HttpResponseBodyMapper responseBodyMapper; 32 35 33 36 @Override 34 37 protected IEntitlementsFactory createInstance() { 35 return new EntitlementsFactory(EntitlementsAPIConfig.builder().rootUrl(config.getAuthorizeAPI()).build()); 38 return new EntitlementsFactory(EntitlementsAPIConfig.builder().rootUrl(config.getAuthorizeAPI()).build(), @a_darafeyeu we are ready to review and merge now.
How does this impact other CSPs? it looks like we are updating method signature in core logic but the issue says it only Azure changes. Can you please clarify?
@wsmatth I see the pipeline is green for AWS. Please approve.
@dkodeih @Nieten @Dmitriy_Rudko @wladmirf Please review
Please register or sign in to reply