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

Global context and its use in Security #2549

Merged
merged 4 commits into from
Dec 2, 2020

Conversation

tomas-langer
Copy link
Member

Resolves #2548

Signed-off-by: Tomas Langer tomas.langer@oracle.com

The Contexts.globalContext() can be used to register components that are application wide (e.g. Singleton) - such as Security, metrics RegistryFactory and Tracer in Helidon MP (in Helidon SE, this is at the discretion of developer, what get registered, if anything).

Updated JAX-RS Client and REST client support for security to use this global registry for cases when we invoke an outbound call without SecurityContext or Context. This is the case for example in messaging flows.

…vailable otherwise

Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
@tomas-langer tomas-langer added security 2.x Issues for 2.x version branch labels Nov 25, 2020
@tomas-langer tomas-langer added this to the 2.2.0 milestone Nov 25, 2020
@tomas-langer tomas-langer self-assigned this Nov 25, 2020
Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Copy link
Member

@spericas spericas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tomas-langer tomas-langer merged commit 5f9e0d7 into helidon-io:master Dec 2, 2020
@tomas-langer tomas-langer deleted the 2548-global-context branch December 2, 2020 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Issues for 2.x version branch security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support outbound security in MP when outside of secured scope
2 participants