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

Filters should be case sensitive or not? #78

Open
leodido opened this issue Mar 19, 2018 · 3 comments
Open

Filters should be case sensitive or not? #78

leodido opened this issue Mar 19, 2018 · 3 comments
Assignees

Comments

@leodido
Copy link
Collaborator

leodido commented Mar 19, 2018

Sorry guys I forgot whether the filters MUST be case sensitive or not.

The following one DOES NOT work.

?filter=username eq "leodido@example.com" AND password eq "3d$h33ran"

The server returns:

{
	"schemas": [
		"urn:ietf:params:scim:api:messages:2.0:Error"
	],
	"status": 400,
	"scimType": "invalidFilter",
	"detail": "Filter is invalid: syntax error at character 33, mismatched input ' ' expecting <EOF>"
}null

(notice the null, there is somewhere a return missing in the error check conditions within the server, too).

Rather the following one works:

?filter=username eq "leodido@example.com" and password eq "3d$h33ran".

@samechelon
Copy link
Collaborator

I believe the error it's given by "AND", it should be lowercase

@leodido
Copy link
Collaborator Author

leodido commented Mar 19, 2018

@samechelon I know the error cause is the uppercase operator. Infact I was asking if it's right that it gives an error or not?

Reformulating: the operators HAVE TO be lowercase by spec?

@alelb
Copy link
Collaborator

alelb commented Mar 19, 2018

Attributes names and attribute operators used in filter are case insensitive (refs). So this is a bug.

@alelb alelb added the bug label Mar 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants