Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

ManageConfig requires additional setSecuritySslContext() method #295

Closed
srinathgit opened this issue Aug 6, 2018 · 2 comments
Closed

ManageConfig requires additional setSecuritySslContext() method #295

srinathgit opened this issue Aug 6, 2018 · 2 comments
Assignees
Milestone

Comments

@srinathgit
Copy link

  1. ManageConfig has two set of credentials; username/password and securityUsername/securityPassword to be used in different circumstances. But there is only on sslContext.
  2. In scenarios where server is set to certificate based authentication, 'sslContext' has to be set to the one generated with a user whose role is a union of 'username' and 'securityUsername' users, else some requests would fail
@rjrudin
Copy link
Contributor

rjrudin commented Aug 7, 2018

What do you propose as a solution for this? A separate e.g. "securitySslContext" field that would be used by ManageClient instead of config.getSslContext?

Should this also be done for hostnameVerifier - i.e. add securityHostnameVerifier?

@srinathgit
Copy link
Author

  1. Yes , a "securitySslContext" field that would be used by ManageClient instead of config.getSslContext is required. Regarding 'securityHostnameVerifier', I had a conversation with @vivekmuniyandi and we think we don't need an additional hostname verifier for 'securityUsername' , the one we already have should suffice.
  2. Additionally, in ManageClient.setManageConfig(ManageConfig config), while creating 'securityUserRestTemplate' , the "securitySslContext" should be used

rjrudin added a commit that referenced this issue Aug 8, 2018
@rjrudin rjrudin added this to the 3.8.3 milestone Aug 8, 2018
@rjrudin rjrudin self-assigned this Aug 8, 2018
@rjrudin rjrudin closed this as completed Aug 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants