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

[Extensions] Generate a password for an extension's service account on installation #2668

Closed
Tracked by #2596
stephen-crawford opened this issue Apr 10, 2023 · 1 comment
Assignees
Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.

Comments

@stephen-crawford
Copy link
Contributor

stephen-crawford commented Apr 10, 2023

After it is possible to read an extension's configuration file and pass that information to the Security plugin, we will need to generate a password for the extension to then use for the creation of its auth header. The password will be used for basic authentication of the service account associated with the extension and should not be stored in the Security plugin. Instead, the password should be immediately hashed and the hash stored as part of the service account details. The password will then be sent back to core where it can be handed to the extension.

Completion of this issue will look like a PR which generates a password for an extension's service account, hashes it, and then stores the hash and sends the password back to core.

@stephen-crawford stephen-crawford converted this from a draft issue Apr 10, 2023
@stephen-crawford stephen-crawford self-assigned this Apr 10, 2023
@stephen-crawford stephen-crawford added the triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable. label Apr 10, 2023
@github-actions github-actions bot added the untriaged Require the attention of the repository maintainers and may need to be prioritized label Apr 10, 2023
@stephen-crawford stephen-crawford moved this from In Progress to Tracking in Security for Extensions Apr 10, 2023
@stephen-crawford stephen-crawford moved this from Tracking to Todo in Security for Extensions Apr 10, 2023
@stephen-crawford stephen-crawford removed the untriaged Require the attention of the repository maintainers and may need to be prioritized label Apr 10, 2023
@stephen-crawford stephen-crawford moved this from Todo to In Progress in Security for Extensions Apr 11, 2023
@stephen-crawford
Copy link
Contributor Author

[Update 4/11] I am working on this in tandem with the token generation issue here. I believe I have found a bearer auth based solution which will avoid requiring passwords for the service accounts. That being said, I am running into some trouble storing the token information for revocation. We don't want to store it in the security index since it is not a configuration file based map. I have not found any other examples in the security plugin which don't use the CType pattern, so working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.
Projects
Status: Done
Development

No branches or pull requests

2 participants