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

Permission or #511

Closed
wants to merge 6 commits into from
Closed

Conversation

huanmeng-qwq
Copy link
Contributor

resolve #429

@Rollczi
Copy link
Owner

Rollczi commented Jan 14, 2025

hey @huanmeng-qwq , thanks for the pullrequest. the solution is too complicated, I cannot accept it immediately.

  • The solution should be mainly contained in the package permissions.
  • StrictService has nothing to do with this.
  • PermissionStrictHandler should not "touch" the platform - I don't even know if we need it since we already have it Set<Set<String>>

I also noticed you have an older branch (from December)

@huanmeng-qwq
Copy link
Contributor Author

#512

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.

Support "OR" behavior for permissions.
2 participants