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
using pattern in acl-file without %c or %u gives unwanted permissions #209
Labels
Milestone
Comments
I'm not sure what the best thing to do here would be. Your original ACL isn't wrong, just not very useful. Perhaps it would be best to produce an error if there isn't a |
ralight
added a commit
that referenced
this issue
Aug 8, 2018
Closes #209. Bug: #209 Signed-off-by: Roger A. Light <[email protected]>
This now rejects patterns that do not contain %c or %u. |
Commit ecb4006 changes the rejection to just emitting a warning. Also see a real world use case discussed in this mailing list thread. |
ralight
added a commit
that referenced
this issue
Nov 8, 2018
Closes #209. Bug: #209 Signed-off-by: Roger A. Light <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I've wrote a acl-file like this:
and created the 2 users "Student" and "Global". I expected, that a anonymous user could only read the topics, but he cloud write to it.
Changing the acl-file to something like this solved the problem:
I've see that I've done a bad mistake, but it confused me, that there was no error or something. So, is it possible to give an error with a wrong acl-file or if better a implementation, that lets "pattern" work like "topic" if there is no %c or %u ?
The text was updated successfully, but these errors were encountered: