Skip to content

[TO rel/0.13] [IOTDB-4018] The path in the permissions contains wildcards * and ** path adaptation does not work#6861

Merged
qiaojialin merged 3 commits intoapache:rel/0.13from
RYH61:add_permission_pathCheck
Aug 14, 2022
Merged

[TO rel/0.13] [IOTDB-4018] The path in the permissions contains wildcards * and ** path adaptation does not work#6861
qiaojialin merged 3 commits intoapache:rel/0.13from
RYH61:add_permission_pathCheck

Conversation

@RYH61
Copy link
Contributor

@RYH61 RYH61 commented Aug 2, 2022

create user, grant insert_timeseries
图片

login new user
图片

login root,grant read_timeseries
图片

login newuser
图片

@RYH61 RYH61 changed the title [IOTDB-4018] The specific path is not verified during authorization [TO rel/0.13] [IOTDB-4018] The specific path is not verified during authorization Aug 2, 2022
@RYH61 RYH61 marked this pull request as draft August 2, 2022 07:13
@RYH61 RYH61 marked this pull request as ready for review August 2, 2022 09:39
@qiaojialin
Copy link
Member

This verification is not needed. We could allow user to insert data on root.**, then users insert into root.sg.d1.s1

@RYH61
Copy link
Contributor Author

RYH61 commented Aug 4, 2022

This verification is not needed. We could allow user to insert data on root.**, then users insert into root.sg.d1.s1

ok

@RYH61 RYH61 force-pushed the add_permission_pathCheck branch 2 times, most recently from e1e48f7 to f9071fb Compare August 4, 2022 08:18
@RYH61 RYH61 marked this pull request as draft August 11, 2022 14:32
@RYH61 RYH61 changed the title [TO rel/0.13] [IOTDB-4018] The specific path is not verified during authorization [TO rel/0.13] [IOTDB-4018] The path in the permissions contains wildcards * and ** path adaptation does not work Aug 11, 2022
@RYH61 RYH61 marked this pull request as ready for review August 11, 2022 16:07
Copy link
Contributor

@SpriCoder SpriCoder left a comment

Choose a reason for hiding this comment

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

Good job! Please check CI.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants