Description

The regex pattern which identifies the "bad characters" for deeplink payload is not correct:

import re

print(f"{ord('A')=}")
print(f"{ord('Z')=}")
print(f"{ord('a')=}")
print(f"{ord('z')=}")
print("-----------------------------------")
for i in range(91, 97):
    print(f"{i}: {chr(i)}")
print("-----------------------------------")
print(re.search(r"[^A-z0-9-]", "hi`bye"))      # None
print(re.search(r"[^A-Za-z0-9-_]", "hi`bye"))  # <re.Match object; span=(2, 3), match='`'>

There is a gap between Z and a. We should explicitly include the ranges A-Z and a-z to avoid accepting characters like backtick.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

I modified the WRONG_PAYLOADS list in tests/test_utils/test_deep_linking.py to demonstrate the bug.

Test Configuration:

• Operating System: MacOS
• Python version: 3.12

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes