-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
*: support require-secure-transport startup option #15341
Conversation
/run-all-tests |
/run-unit-test |
nmPluginLoad = "plugin-load" | ||
nmRepairMode = "repair-mode" | ||
nmRepairList = "repair-list" | ||
nmRequireSecureTransport = "require-secure-transport" |
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.
I know it is the same name as MySQL's, but I'm not sure adding it is approved.
\cc @siddontang
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.
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.
I use mysql --help
but don't find hat it uses this in command flags.
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.
It is a flag of mysqld
, try mysqld --verbose --help
Codecov Report
@@ Coverage Diff @@
## master #15341 +/- ##
===============================================
+ Coverage 80.532% 80.6704% +0.1383%
===============================================
Files 502 503 +1
Lines 134534 135968 +1434
===============================================
+ Hits 108343 109686 +1343
- Misses 17765 17845 +80
- Partials 8426 8437 +11 |
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
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
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-3.0 in PR #15415 |
What problem does this PR solve?
fixes #15306
What is changed and how it works?
--require-secure-transport
startup optionrequire-secure-transport
be enabledrequire-secure-transport
be enabled but no avaliable cert/key/carequire-secure-transport
be enabled event if withno rollback on error
Check List
Tests
Code changes
Side effects
Related changes
tidb-ansible
repositoryRelease note
This change is