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.

Sign up for GitHub

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

Jump to bottom

Decouple OCI authentication from CDI #7327

Closed
ljnelson opened this issue Aug 7, 2023 · 4 comments
Closed

Decouple OCI authentication from CDI #7327

ljnelson opened this issue Aug 7, 2023 · 4 comments
Assignees
@ljnelson
Labels
3.x Issues for 3.x version branch cdi CDI OCI P2

Comments

@ljnelson
Copy link
Member

ljnelson commented Aug 7, 2023

The convenient OCI authentication mechanism that currently underlies the OCI extension is already CDI-independent, but lives in a CDI-centric package. I need to move it out to its own place, refactoring and cleaning up as necessary, and then need to ensure the OCIExtension depends on it properly. Ideally there should be no change in behavior or configuration.

Part of the effort for #4238.
@ljnelson ljnelson added P2 cdi CDI 3.x Issues for 3.x version branch OCI labels Aug 7, 2023
@ljnelson ljnelson self-assigned this Aug 7, 2023
@ljnelson ljnelson linked a pull request Aug 8, 2023 that will close this issue
Adds AdpSuppliers under oci/sdk/sdk #7331
Closed
@ljnelson
Copy link
Member Author

Pausing as this is being done elsewhere

trentjeff added a commit to trentjeff/helidon that referenced this issue Aug 11, 2023
@trentjeff
In support of helidon-io#7327
c416d40
@ljnelson
Copy link
Member Author

Closing; will not complete

@ljnelson
Copy link
Member Author

See also: #7373

@ljnelson
Copy link
Member Author

See also: #7352

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ljnelson ljnelson

Labels
3.x Issues for 3.x version branch cdi CDI OCI P2
Projects
Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds AdpSuppliers under oci/sdk/sdk ljnelson/helidon
1 participant
@ljnelson