Fix sanitizeUrl signature for advent controller #728

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

Merged

CySSoO merged 1 commit into master from cyssoo/fix-method-signature-compatibility-issue

Dec 11, 2025

Contributor

CySSoO commented Dec 11, 2025

Summary

align advent front controller sanitizeUrl signature with FrontControllerCore
update URL sanitization to return strings while preserving safety checks
adjust window payload handling to treat empty sanitized URLs as invalid

Testing

php -l controllers/front/advent.php


          Fix sanitizeUrl signature for advent controller

dc1d05b

CySSoO added the codex label

— with

ChatGPT Codex Connector

CySSoO merged commit 228e663 into master

2 of 6 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels