Skip to content

Fix access rights required for the merge() table function.#17983

Merged
vitlibar merged 1 commit intoClickHouse:masterfrom
vitlibar:fix-access-rights-for-merge-table-function
Dec 15, 2020
Merged

Fix access rights required for the merge() table function.#17983
vitlibar merged 1 commit intoClickHouse:masterfrom
vitlibar:fix-access-rights-for-merge-table-function

Conversation

@vitlibar
Copy link
Copy Markdown
Member

@vitlibar vitlibar commented Dec 10, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category:

  • Improvement

Changelog entry:
Now the table function merge() requires the current user to have the SELECT privilege on each table it receives data from. This PR fixes #16964

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Dec 10, 2020
@vitlibar vitlibar force-pushed the fix-access-rights-for-merge-table-function branch 5 times, most recently from 2ca4d24 to cec9b38 Compare December 11, 2020 13:04
@alexey-milovidov
Copy link
Copy Markdown
Member

Merged prematurally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-docs-needed pr-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RBAC - User with SELECT and CREATE TABLE on table1 can access table2 without SELECT on table2 using merge()

4 participants