Skip to content

Commit

Permalink
%<alnum> is the start of a function - so use %% for literal %
Browse files Browse the repository at this point in the history
  • Loading branch information
ndptech committed Dec 18, 2023
1 parent 8a7c32c commit c7fbb63
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/tests/modules/ldap/map.unlang
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ if (!notfound) {
test_fail
}

map ldapldapi "ldapi://%2Ftmp%2Fldap%2Fsocket/ou=people,dc=example,dc=com??sub?(uid=John)" {
map ldapldapi "ldapi://%%2Ftmp%%2Fldap%%2Fsocket/ou=people,dc=example,dc=com??sub?(uid=John)" {

This comment has been minimized.

Copy link
@alandekok

alandekok Dec 18, 2023

Member

I would argue that the parser should be smart enough to see %[0-9a-fA-F]+ and then not a (, and just treat % as %.

But it's a hard thing to get right.

&Tmp-String-2 := 'displayName'
}

Expand Down

0 comments on commit c7fbb63

Please sign in to comment.