Skip to content
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

Store hashed credential identities #6

Closed
ryanemerson opened this issue Aug 30, 2019 · 2 comments
Closed

Store hashed credential identities #6

ryanemerson opened this issue Aug 30, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@ryanemerson
Copy link
Collaborator

Currently we store key/values in plain text as it's necessary for our probe implementations to authenticate with the rest endpoint in order to retrieve health information. Once infinispan/infinispan#7248 has been merged, it will be possible to request this information anonymously, therefore we should no longer store credentials in plain-text.

@ryanemerson ryanemerson added the enhancement New feature or request label Aug 30, 2019
@ryanemerson ryanemerson changed the title Store hashed credential identies Store hashed credential identities Aug 30, 2019
ryanemerson added a commit to infinispan/infinispan-image-artifacts that referenced this issue Oct 10, 2019
ryanemerson added a commit to infinispan/infinispan-image-artifacts that referenced this issue Oct 14, 2019
@galderz
Copy link
Member

galderz commented Oct 15, 2019

I think this means we can get rid of the operator user we were using for the probes, is that right? /cc @rigazilla

@rigazilla
Copy link
Contributor

I think this means we can get rid of the operator user we were using for the probes, is that right? /cc @rigazilla

@galderz infinispan/infinispan#7248 allows unauth access for health and ready probes, but operator needs to call other API resources: cluster info, graceful shutdown and maybe others... that can't be unauthenticated.

We still need a kind of authenticated way for the operator/infinispan interaction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants