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.

Sign up for GitHub

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

Jump to bottom

Execution token character limitations, need warning message. #29

Closed
lat9 opened this issue Feb 7, 2024 · 0 comments
Closed

Execution token character limitations, need warning message. #29

lat9 opened this issue Feb 7, 2024 · 0 comments
Labels
change New feature or request fix provided A correction has been provided
Milestone
v4.0.0

Comments

@lat9
Copy link
Owner

lat9 commented Feb 7, 2024

... since the core init_sanitize.php strips all but alphanumerics and these additional characters: _.-. See the following blob for details.

https://github.com/zencart/zencart/blob/a149d1fbf47ca869df02ff1b275eb1066dc5a2f6/includes/init_includes/init_sanitize.php#L155-L187
@lat9 lat9 added the change New feature or request label Feb 7, 2024
@lat9 lat9 added this to the v4.0.0 milestone Feb 7, 2024
lat9 added a commit that referenced this issue Feb 7, 2024
@lat9
#29: Add error message when token has invalid characters
37a25f7
@lat9 lat9 added the fix provided A correction has been provided label Feb 7, 2024
lat9 added a commit that referenced this issue Feb 8, 2024
@lat9
#29: Correcting update ...
f213883 
... since it's no longer allowing an empty token (sigh).
@lat9 lat9 closed this as completed Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
change New feature or request fix provided A correction has been provided
Projects
None yet
Milestone
v4.0.0
Development

No branches or pull requests
1 participant
@lat9