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

Backslash escape visible before underscore for user accounts area #13614

Closed
ibennetch opened this issue Aug 24, 2017 · 2 comments
Closed

Backslash escape visible before underscore for user accounts area #13614

ibennetch opened this issue Aug 24, 2017 · 2 comments
Assignees
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
Milestone

Comments

@ibennetch
Copy link
Member

I have a database and table with an underscore in the name, such as 'a_z'.

When editing user privileges or creating a new user, I can go to the "Database" tab to edit the database-level permissions for that user. After selecting the database from the list of available database, I'm taken to the screen to edit permissions.

In the header across the top:
Edit privileges: User account 'a'@'%' - Database a\_z

The hint at the very bottom:
[ Database a\_z: Structure ]
By the way, that link is broken (clicking it gives an error "No databases selected" which I'm guessing is related.

I don't want to see the backslash. It's not really part of my database name, so it shouldn't be shown like this.

If I go further to the table-level privileges, I do not see the backslash escape character.

This is tested with current master

@madhuracj
Copy link
Contributor

madhuracj commented Aug 29, 2017

Since '_' is a wildcard for privileges, '_' in database names are escaped to make the distinction between the two. However, I agree that this is inconsistent with the headers of table-level privileges.

madhuracj added a commit that referenced this issue Aug 29, 2017
Signed-off-by: Madhura Jayaratne <madhura.cj@gmail.com>
@williamdes williamdes added the ui Issues relating to the user interface label Oct 28, 2018
@williamdes williamdes added this to Triage zone in Enhancements Jan 23, 2020
@williamdes williamdes added this to Needs triage in issues via automation Jan 24, 2020
@williamdes williamdes removed this from Triage zone in Enhancements Jan 24, 2020
@williamdes williamdes moved this from Needs triage to n/a priority in issues Jan 24, 2020
@williamdes williamdes self-assigned this Jul 28, 2021
@williamdes williamdes added this to the 5.1.2 milestone Jul 28, 2021
@williamdes williamdes removed their assignment Jul 28, 2021
@williamdes williamdes added the Bug A problem or regression with an existing feature label Jul 28, 2021
@williamdes
Copy link
Member

cc @iifawzi
I can confirm that using an escaped DB name and then trying to grant a table will forward you to a page that has no escaping and the link of the db name (not escaped) will lead you to a non working page (because of missing escaping chars)

@williamdes williamdes moved this from n/a priority to to be fixed soon in issues Jul 28, 2021
@williamdes williamdes added the has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete label Jul 28, 2021
@williamdes williamdes self-assigned this Jul 28, 2021
williamdes added a commit that referenced this issue Jul 28, 2021
Signed-off-by: William Desportes <williamdes@wdes.fr>
issues automation moved this from to be fixed soon to Closed Jul 29, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
issues
  
Closed
Development

No branches or pull requests

3 participants