Skip to content

LiteSpeed: document security considerations for executed paths - #24450

Merged
DanielEScherzer merged 1 commit into
php:PHP-8.6from
DanielEScherzer:lsapi-script-path-security
Oct 26, 2026
Merged

DanielEScherzer merged 1 commit into
php:PHP-8.6from
DanielEScherzer:lsapi-script-path-security

Conversation

@DanielEScherzer

Copy link
Copy Markdown
Member

Make it clear that the init_request_info() function is indeed meant to be using LSAPI_GetScriptFileName() without any validation, and document in the README file that this lack of validation is another reason why only trusted sources should be granted direct access to the listener.

@DanielEScherzer DanielEScherzer added Category: Documentation This is for documentation inside php-src, not on php.net SAPI: litespeed labels Oct 19, 2026
Make it clear that the `init_request_info()` function is indeed meant to be
using `LSAPI_GetScriptFileName()` without any validation, and document in the
README file that this lack of validation is another reason why only trusted
sources should be granted direct access to the listener.
@DanielEScherzer
DanielEScherzer force-pushed the lsapi-script-path-security branch from 875dae5 to fbe0f21 Compare October 23, 2026 01:23
@DanielEScherzer

Copy link
Copy Markdown
Member Author

As this is a documentation only change, I plan to merge it in a few days if there are no objections

@DanielEScherzer
DanielEScherzer merged commit e5f67fb into php:PHP-8.6 Oct 26, 2026
18 checks passed
@DanielEScherzer
DanielEScherzer deleted the lsapi-script-path-security branch October 26, 2026 21:33
DanielEScherzer added a commit that referenced this pull request Oct 26, 2026
* PHP-8.6:
  LiteSpeed: document security considerations for executed paths (#24450)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Documentation This is for documentation inside php-src, not on php.net SAPI: litespeed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant