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.

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

privilege: fix show grants privilege check #15524

Merged
merged 6 commits into from
Mar 23, 2020

Conversation

imtbkcat
Copy link

What problem does this PR solve?

SHOW GRANTS need SELECT privilege on mysql database, but tidb dosen't check.

What is changed and how it works?

add privilege check for show grants

Related changes

  • Need to cherry-pick to the release branch

Check List

Tests

  • Unit test

Side effects

  • Breaking backward compatibility

Release note

fix SHOW GRANTS privilege check

@imtbkcat imtbkcat requested a review from a team as a code owner March 20, 2020 10:41
@ghost ghost requested review from qw4990 and XuHuaiyu and removed request for a team March 20, 2020 10:41
@github-actions github-actions bot added the sig/execution SIG execution label Mar 20, 2020
@codecov
Copy link

codecov bot commented Mar 20, 2020

Codecov Report

Merging #15524 into master will increase coverage by 0.0111%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##             master    #15524        +/-   ##
===============================================
+ Coverage   80.4029%   80.414%   +0.0111%     
===============================================
  Files           503       503                
  Lines        134408    134423        +15     
===============================================
+ Hits         108068    108095        +27     
+ Misses        17838     17836         -2     
+ Partials       8502      8492        -10

Copy link
Member

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

@tiancaiamao
Copy link
Contributor

LGTM

tiancaiamao
tiancaiamao previously approved these changes Mar 23, 2020
@tiancaiamao
Copy link
Contributor

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Mar 23, 2020

Your auto merge job has been accepted, waiting for 15534

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label Mar 23, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Mar 23, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Mar 23, 2020

@imtbkcat merge failed.

@imtbkcat imtbkcat added the priority/release-blocker This issue blocks a release. Please solve it ASAP. label Mar 23, 2020
@imtbkcat
Copy link
Author

/run-all-tests

@imtbkcat
Copy link
Author

/run-all-tests

@imtbkcat
Copy link
Author

/run-all-tests tidb-test=pr/1009

@imtbkcat
Copy link
Author

/run-common-test tidb-test=pr/1009

@imtbkcat
Copy link
Author

/run-integration-common-test tidb-test=pr/1009

@tiancaiamao
Copy link
Contributor

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Mar 23, 2020

/run-all-tests

@sre-bot sre-bot merged commit b8ea8fc into pingcap:master Mar 23, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Mar 23, 2020

cherry pick to release-3.0 in PR #15601

sre-bot pushed a commit to sre-bot/tidb that referenced this pull request Mar 23, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot
Copy link
Contributor

sre-bot commented Mar 23, 2020

cherry pick to release-3.1 in PR #15602

imtbkcat pushed a commit to imtbkcat/tidb that referenced this pull request Mar 24, 2020
imtbkcat pushed a commit to imtbkcat/tidb that referenced this pull request Mar 31, 2020
imtbkcat pushed a commit to imtbkcat/tidb that referenced this pull request Apr 8, 2020
imtbkcat pushed a commit to sre-bot/tidb that referenced this pull request Apr 13, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/privilege priority/release-blocker This issue blocks a release. Please solve it ASAP. sig/execution SIG execution status/can-merge Indicates a PR has been approved by a committer. type/bugfix This PR fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants