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.

Sign up for GitHub

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

Jump to bottom

[Bug] [admin] Task permission aspect do not work when enable 'OWNER' startegy #4102

Closed
3 tasks done
MactavishCui opened this issue Dec 30, 2024 · 1 comment · Fixed by #4103
Closed
3 tasks done

[Bug] [admin] Task permission aspect do not work when enable 'OWNER' startegy #4102

MactavishCui opened this issue Dec 30, 2024 · 1 comment · Fixed by #4103
Labels
Bug Something isn't working Waiting for reply Waiting for reply

Comments

@MactavishCui
Copy link
Contributor

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

When enable strategy not equals 'ALL', although task operation buttons is disabled, tasks can still be operated by restful interfaces as follows:
For example:
here is a task belongs to admin
d1f891fb42b3676486bdda92dc63de2
other users can still operate task by interfaces
a55547ed6a4ed0e75dd34e493dac8bd

What you expected to happen

Fix this bug

How to reproduce

call interface

Anything else

The reason of this bug is this:
0242cb825100dfd313da70facbed911

Version

dev

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@MactavishCui MactavishCui added Bug Something isn't working Waiting for reply Waiting for reply labels Dec 30, 2024
@MactavishCui
Copy link
Contributor Author

MactavishCui commented Dec 30, 2024 •
edited
Loading

I have submitted a PR to fix this bug: #4103

@MactavishCui MactavishCui changed the title [Bug] [dinky-admin] Task permission aspect do not work when enable 'OWNER' startegy [Bug] [admin] Task permission aspect do not work when enable 'OWNER' startegy Dec 30, 2024
@Zzm0809 Zzm0809 linked a pull request Dec 31, 2024 that will close this issue
[Bug-4102][admin]Bug fix for task lock strategy condition #4103
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something isn't working Waiting for reply Waiting for reply
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Bug-4102][admin]Bug fix for task lock strategy condition MactavishCui/dinky
1 participant
@MactavishCui