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

Browse fails with users who have only column privileges for some columns #11922

Closed
ibennetch opened this Issue Feb 6, 2016 · 6 comments

Comments

Projects
None yet
4 participants
@ibennetch
Copy link
Member

ibennetch commented Feb 6, 2016

Users who only have select privileges on some of the columns in a table are unable to browse.

Is this something we can fix?

SQL to make reproducing easier:

GRANT USAGE ON *.* TO 'col'@'%';

GRANT SELECT (name, id), INSERT (name, id), UPDATE (name, id), REFERENCES (name, id) ON `test`.`col` TO 'col'@'%';

and to create the table:

CREATE TABLE `col` (
  `id` int(11) NOT NULL,
  `name` varchar(255) NOT NULL,
  `email` varchar(255) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1;
@ibennetch

This comment has been minimized.

@madhuracj

This comment has been minimized.

Copy link
Member

madhuracj commented Feb 11, 2016

Given that only a very small fraction of users would have the problem, I am not sure checking column permission prior to generating the browse query is worth performance burden it adds to all the other users. Moreover, using a bookmark having the same name as the table with the appropriate browse query provides a workaround for this.

@madhuracj

This comment has been minimized.

Copy link
Member

madhuracj commented Feb 11, 2016

@madhuracj madhuracj removed their assignment Feb 11, 2016

@ibennetch ibennetch self-assigned this Feb 11, 2016

@ibennetch

This comment has been minimized.

Copy link
Member Author

ibennetch commented Feb 11, 2016

I don't see a good solution to this.

I'm interested in the bookmark workaround you mentioned and think it's acceptable to just document this as a known issue.

@madhuracj madhuracj added this to the 4.5.5 milestone Feb 14, 2016

@madhuracj

This comment has been minimized.

Copy link
Member

madhuracj commented Feb 14, 2016

Addressed by PR #11938

@madhuracj madhuracj closed this Feb 14, 2016

@freshcalendars

This comment has been minimized.

Copy link

freshcalendars commented Jan 7, 2019

You helped me solve a big problem. Thank you so much.

xoxo 😋

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.