Skip to content

Add support for fieldnames starting with dollar-sign like $ref#6

Open
kapersoft wants to merge 1 commit intoCloudstek:masterfrom
kapersoft:master
Open

Add support for fieldnames starting with dollar-sign like $ref#6
kapersoft wants to merge 1 commit intoCloudstek:masterfrom
kapersoft:master

Conversation

@kapersoft
Copy link
Copy Markdown

@kapersoft kapersoft commented May 30, 2024

In my project a patch request is received from a third-party tool where the $ref-field is used to select the group member for removal:

{
    "schemas":
    [
        "urn:ietf:params:scim:api:messages:2.0:PatchOp"
    ],
    "Operations":
    [
        {
            "op": "remove",
            "path": "members[$ref eq \"https://example.com/v2/Users/d46e22f1-74ab-4179-a576-a441a136a9d9\"]"
        }
    ]
}

The parser cannot parse this property and throws the following exception: Nette\Tokenizer\Exception: Unexpected '$ref eq "h' on line 1, column 9.

This PR fixes this issue by updating the T_NAME-regex. A test is added as prove.

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.

1 participant