Skip to content

PHOENIX-7976: Introduce new parameters to support single-purpose Extended Key Usage (EKU) certificates - #194

Merged
richardantal merged 1 commit into
apache:masterfrom
PDavid:PHOENIX-7976
Sep 2, 2026
Merged

richardantal merged 1 commit into
apache:masterfrom
PDavid:PHOENIX-7976

Conversation

@PDavid

@PDavid PDavid commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

To maintain compatibility assume that the current parameters are still Dual-purpose EKU certs and introduce new parameters. Fall back to the old parameters if the new one is not available.

Current params:
"phoenix.queryserver.tls.keystore";
"phoenix.queryserver.tls.truststore.password";
"phoenix.queryserver.tls.keystore.password";
"phoenix.queryserver.tls.truststore";

New proposed parameters:
"phoenix.queryserver.tls.server.keystore";
"phoenix.queryserver.tls.server.keystore.password"; "phoenix.queryserver.tls.server.truststore";
"phoenix.queryserver.tls.server.truststore.password";

…nded Key Usage (EKU) certificates

To maintain compatibility assume that the current parameters are still Dual-purpose EKU certs and introduce new parameters.
Fall back to the old parameters if the new one is not available.

Current params:
"phoenix.queryserver.tls.keystore";
"phoenix.queryserver.tls.truststore.password";
"phoenix.queryserver.tls.keystore.password";
"phoenix.queryserver.tls.truststore";

New proposed parameters:
"phoenix.queryserver.tls.server.keystore";
"phoenix.queryserver.tls.server.keystore.password";
"phoenix.queryserver.tls.server.truststore";
"phoenix.queryserver.tls.server.truststore.password";

@richardantal richardantal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@richardantal
richardantal merged commit b8a4a44 into apache:master Sep 2, 2026
1 check passed
@PDavid
PDavid deleted the PHOENIX-7976 branch September 2, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants