Skip to content

Conversation

@robbavey
Copy link
Member

@robbavey robbavey commented Dec 2, 2024

This commit marks the following SSL settings as obsolete:

ssl_cert, which should be replaced by ssl_certificate
ssl_enable, which should be replaced by ssl_enabled
ssl_verify, which should be replaced by ssl_client_authentication when mode is server or ssl_verification_modewhen mode is client

Relates: #225

This commit marks the following SSL settings as obsolete:

`ssl_cert`, which should be replaced by `ssl_certificate`
`ssl_enable`, which should be replaced by `ssl_enabled`
`ssl_verify`, which should be replaced by `ssl_client_authentication` when `mode` is `server` or `ssl_verification_mode`when mode is `client`
Copy link
Contributor

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

Just a couple small docs comments. Code removal looks clean.

Copy link
Contributor

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

Assuming CI is green i think this is good! Apologies for not catching everything on the first take 😬

@robbavey
Copy link
Member Author

Assuming CI is green i think this is good! Apologies for not catching everything on the first take 😬

All good! Thanks for being thorough and catching all of my 🤦 ...

@robbavey robbavey requested a review from karenzone December 10, 2024 21:11
@robbavey
Copy link
Member Author

Ready for doc review @karenzone

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Updated text to align with messaging for other plugins

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Suggestion inline

Co-authored-by: Karen Metts <[email protected]>
@robbavey
Copy link
Member Author

robbavey commented Jan 3, 2025

Ready for another round @karenzone

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

LGTM

@robbavey robbavey merged commit 5977fcc into main Jan 10, 2025
3 checks passed
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.

4 participants