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

Review all secrets across all DCAs #375

Open
ychae opened this issue Jul 25, 2022 · 3 comments
Open

Review all secrets across all DCAs #375

ychae opened this issue Jul 25, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@ychae
Copy link
Collaborator

ychae commented Jul 25, 2022

Describe the bug
Need to review all secrets and identify:

  1. Time out of secrets and how do we update them
  2. Who owns them (e.g. service accounts)
  3. What types of secrets are currently stored

This is likely the cause of users logging in as themselves and seeing someone else's username in the dashboard

To Reproduce
Steps to reproduce the behavior:
Unknown yet, haven't been able to reliably reproduce the error

Expected behavior
Need to make sure that all secrets remain up to date and correct. And also ensure that the username that shows up on the dashboard is the same as the username of the person that has logged in.

Priority (select one)
Major 📢

Screenshots

During demo testing, @milen-sage and I were testing the DCA dashboard, this is a screenshot of @milen-sage's view where he was logged into his account but saw my username in the dashboard instead.

Screen Shot 2022-07-21 at 4 33 20 PM

Additional context
Add any other context about the problem here.

@ychae ychae added the bug Something isn't working label Jul 25, 2022
@afwillia
Copy link
Contributor

I just logged into the data curator demo and saw my username in the dashboard as expected 🤔

@ychae
Copy link
Collaborator Author

ychae commented Jul 25, 2022

We've noticed that it tends to happen if two folks are using the same app at the same time, but we couldn't identify the exact sequence to get it to repro 😭

@milen-sage
Copy link
Contributor

Related to #341; partially redundant but has a specific instance of secret mishandling (likely resolved in latest releases of DCA, but will keep open to confirm).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants