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.

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

[BUGFIX] Avoid PHP warning for request of eID=FalSecuredownloadFileTreeState #216

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

sgrossberndt
Copy link
Contributor

Avoid "PHP Warning: Undefined array key "folder" in Classes/Controller/FileTreeStateController.php" (and subsequent creation of sys_log record) when requesting /index.php?eID=FalSecuredownloadFileTreeState without a "folder" parameter by defining a default value.

As the same issue might arise for a call to an existing folder requesting without an "open" parameter define a default here too.

These calls usually are triggered by bots calling all URLs found in the site Javascript.

…nloadFileTreeState

Avoid "PHP Warning: Undefined array key "folder" in Classes/Controller/FileTreeStateController.php" (and subsequent creation of sys_log record) when requesting /index.php?eID=FalSecuredownloadFileTreeState without a "folder" parameter by defining a default value.

As the same issue might arise for a call to an existing folder requesting without an "open" parameter define a default here too.

These calls usually are triggered by bots calling all URLs found in the site Javascript.
@sgrossberndt
Copy link
Contributor Author

As this is the only issue found so far for the production use, would it be possible to tag a new version 5.0.0 when this has been merged? Thanks!

@joey-bolts joey-bolts merged commit ef733be into beechit:master Sep 12, 2023
@joey-bolts
Copy link
Collaborator

@sgrossberndt Release made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants