Skip to content
Snippets Groups Projects

adds support azure multi-tenancy

Merged Aliaksei Darafeyeu requested to merge azure_multi_tenancy into master

Type of change

  • Feature

#22 (closed)

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

Merge request pipeline #27214 failed

Pipeline: Notification

#27220

    Merge request pipeline failed for c849862b

    Deployed to A‎WS‎ 4 years ago
    Deployed to I‎BM‎ 4 years ago

    Merged by ethiraj krishnamanaiduethiraj krishnamanaidu 4 years ago (Feb 18, 2021 8:04pm UTC)

    Loading

    Pipeline #27274 failed

    Pipeline failed for a22668fb on master

    Deployed to I‎BM‎ 4 years ago
    Deployed to A‎WS‎ 4 years ago

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • added 1 commit

    Compare with previous version

  • added 4 commits

    Compare with previous version

  • added 1 commit

    • d4a51be0 - replaces CredentialHeadersProvider with NotificationFilter

    Compare with previous version

  • added 21 commits

    Compare with previous version

  • added 1 commit

    • 848bf359 - adds support azure multi-tenancy

    Compare with previous version

  • added 1 commit

    • e19e15c0 - adds support azure multi-tenancy

    Compare with previous version

  • added 8 commits

    Compare with previous version

  • Aliaksei Darafeyeu marked this merge request as ready

    marked this merge request as ready

  • Aliaksei Darafeyeu resolved all threads

    resolved all threads

  • Duvelis Carao approved this merge request

    approved this merge request

  • 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());
  • Komal Makkar approved this merge request

    approved this merge request

  • added 4 commits

    Compare with previous version

  • Aliaksei Darafeyeu changed the description

    changed the description

  • 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(),
  • Aliaksei Darafeyeu changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading