fabbricadigitale / scimd

SCIM v2 golang implementation
MIT License
5 stars 1 forks source link

Canonical form of "not" operator in filtering syntax #21

Open leogr opened 7 years ago

leogr commented 7 years ago

RFC7644 is ambiguous about the not trailing space.

ABNF rule say: FILTER = attrExp / logExp / valuePath / *1"not" "(" FILTER ")" (ie. no trailing space)

But examples have spaces:

filter=userType ne "Employee" and not (emails co "example.com" or emails.value co "example.org")

Currently, the implementation includes a trailing space thus we're assuming that not () is the canonical form, even the parser matches both not () and not().

Maybe it's not problematic.