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

Determine how auth tokens are forwarded to Extensions #2764

Closed
Tracked by #2573
davidlago opened this issue May 11, 2023 · 5 comments
Closed
Tracked by #2573

Determine how auth tokens are forwarded to Extensions #2764

davidlago opened this issue May 11, 2023 · 5 comments
Assignees
Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.

Comments

@davidlago
Copy link

With the security plugin, the auth token to be forwarded to an extension will be generated after successful authentication on a request destined for an extension. The actual request forwarding is performed in the extensions rest handler RestSendToExtensionAction. This handler needs to be able to obtain the generated token to forward to the extension.

@github-actions github-actions bot added the untriaged Require the attention of the repository maintainers and may need to be prioritized label May 11, 2023
@stephen-crawford
Copy link
Contributor

We can use this: opensearch-project/OpenSearch#7452. The security plugin would implement the interface and then the Extension manager could pass everything into the RestSendToExtensionAction.

@cwperks cwperks added triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable. and removed untriaged Require the attention of the repository maintainers and may need to be prioritized labels May 15, 2023
@cwperks
Copy link
Member

cwperks commented May 15, 2023

[Triage] Assigned to @scrawfor99 . Please update with more details on what the initial interface will look like.

@stephen-crawford
Copy link
Contributor

Overview of the interface:

PR opensearch-project/OpenSearch#7452, will introduce a new TokenManager interface to be used as a template for what an IdentityPlugin will expect any implementation to support. The TokenManager Interface defines five operations which are supposed to be implemented:

  1. public AuthToken issueToken(): Issues a new AuthToken
  2. public boolean validateToken(AuthToken token): Validates an auth token based on the rules associated with its format
  3. public String getTokenInfo(AuthToken token): Fetch the info from a token
  4. public void revokeToken(AuthToken token): Revokes a token that should no longer be treated as valid
  5. public void resetToken(AuthToken token): Updates a token to be valid for a greater period of time or to have different attributes.

Each of the implementing methods are relatively straightforward. The only ones which may require complicated handling are token issuance and token revocation.

Issuing a token is likely to requiring overloading the method in some implementations of the IdentityPlugin. For example you may need to pass extension information to an IdentityPlugin for a new token to be created for that extension. At the same time, a key-based system could simply generate a random key with no information.

Revoking a token can also be complicated depending on the implementation. For example, you would not want to create a revocation list for a token that encoded BasicAuth information. Instead, you may need to erase the token to effectively disable it.

@davidlago davidlago changed the title Determine how auth token is generated from security plugin and accessible by core to forward Determine how auth tokens are forwarded to Extensions Jul 13, 2023
@stephen-crawford
Copy link
Contributor

opensearch-project/OpenSearch#8679

@cwperks
Copy link
Member

cwperks commented Jul 13, 2023

@scrawfor99 Is there a companion security PR?

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

4 participants