-
Notifications
You must be signed in to change notification settings - Fork 2
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
Regex match logic: Rust is_match() vs. JDK matches() #31
Labels
bug
Something isn't working
Comments
35VLG84
added a commit
to e257-fi/tackler-t3db
that referenced
this issue
Dec 20, 2024
e257-fi/tackler-ng#31 Signed-off-by: 35V LG84 <35vlg84-x4e6b92@e257.fi>
35VLG84
added a commit
to e257-fi/tackler-tests
that referenced
this issue
Dec 20, 2024
test: 5c34d752-8d17-40df-be91-5dc1b107478e e257-fi/tackler-ng#31 Signed-off-by: 35V LG84 <35vlg84-x4e6b92@e257.fi>
35VLG84
changed the title
Account Selector match logic: rust is_match() vs. JDK matches()
Regex match logic: Rust is_match() vs. JDK matches()
Dec 20, 2024
So the Rust regex must be anchored to have the same behavior as with JDK
The Regex and RegexSets have to be peeled away from that wrapper before calculating the checksum. |
35VLG84
added a commit
to e257-fi/tackler-tests
that referenced
this issue
Dec 21, 2024
t3db: 5c34d752-8d17-40df-be91-5dc1b107478e e257-fi/tackler-ng#31 Signed-off-by: 35V LG84 <35vlg84-x4e6b92@e257.fi>
35VLG84
added a commit
that referenced
this issue
Dec 22, 2024
GH-31 Signed-off-by: 35V LG84 <35vlg84-x4e6b92@e257.fi>
35VLG84
added a commit
to e257-fi/tackler-tests
that referenced
this issue
Dec 22, 2024
t3db: 5c34d752-8d17-40df-be91-5dc1b107478e, 85d16b5a-cde9-40d3-9a37-3b7ba7ee7049 e257-fi/tackler-ng#31 Signed-off-by: 35V LG84 <35vlg84-x4e6b92@e257.fi>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
JDK
Rust
Rust examples:
This will make Account Selectors behave differently, and on the other hand, the fix (
^pattern$
) will change Account Selector Checksums.The text was updated successfully, but these errors were encountered: