-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Document additional credential providers for JDBC connectors #14454
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
Conversation
|
@Jessie212 @mosabua maybe you could take a look at this one |
|
I think this is a good addition to the docs. |
55a30c9 to
809b011
Compare
|
This supersedes #11791 |
809b011 to
679fddb
Compare
mosabua
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor improvements needed.. then good to go.
679fddb to
7e52dd6
Compare
hashhar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from me.
@mosabua please approve if it looks good to you as well and I'll merge.
mosabua
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go!
|
Please note that |
Description
Document credential providers introduced in #1124 and #1521
Non-technical explanation
n/a
Release notes
(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:
Fixes #1910, #1911