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

simplified domain of admins capability is the empty string #222

Open
ael-code opened this issue Nov 13, 2015 · 7 comments
Open

simplified domain of admins capability is the empty string #222

ael-code opened this issue Nov 13, 2015 · 7 comments
Labels

Comments

@ael-code
Copy link
Member

regular domain is .+ (match all).
simToReg(".+") is not capable to handle this string and return the empty string.

@ael-code ael-code added the auth label Nov 13, 2015
@boyska
Copy link
Member

boyska commented Dec 3, 2015

have you any clue about why can't simToReg handle it?

@ael-code
Copy link
Member Author

have you any clue about why can't simToReg handle it?

Because it's not a simplified regular expression.

@boyska
Copy link
Member

boyska commented Dec 27, 2015

I can't understand the consequences of this bug

@ael-code
Copy link
Member Author

We need to extend our simplified regular expression in order to support multilevel expansion. At the moment we only support special char * that is an intra-slashes expansion.

simToReg will convert * in [^/]+.
We need another special char to be converted in .+

@boyska
Copy link
Member

boyska commented Dec 27, 2015

no, really, I am missing something. Is there any bug? are there thing that do not work? are there things that a user cannot do?

@ael-code
Copy link
Member Author

On 12/27/2015 04:01 PM, BoySka wrote:

no, really, I am missing something. Is there any bug? are there thing
that do not work? are there things that a user cannot do?

It's not possible to create a capability with a domain that match all
strings after certain char.

For instance it's not possible to generate some domain like '.+' or
'^/?volumes/.+' passing through a simplified expression.

@boyska
Copy link
Member

boyska commented Dec 27, 2015

ael wrote:

On 12/27/2015 04:01 PM, BoySka wrote:

no, really, I am missing something. Is there any bug? are there thing
that do not work? are there things that a user cannot do?

It's not possible to create a capability with a domain that match all
strings after certain char.

For instance it's not possible to generate some domain like '.+' or
'^/?volumes/.+' passing through a simplified expression.

oh, ok, now this is much clearer.

Why don't we fix at the UI layer? I mean, if a user wants to have
simplified expression (99% of the time that's what she wants) then
everything is fine. Clicking on a certain checkbox, normal regexp can be
inserted directly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants