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
SecurityTest::testGetURLContent fails - ftp connection is allowed despite default options #28168
Labels
Bug
This is a bug (something does not work as expected)
Comments
mdeweerd
changed the title
Security test fails:
SecurityTest::testGetURLContent fails - ftp connection is allowed despite default options
Feb 14, 2024
mdeweerd
added a commit
to mdeweerd/dolibarr
that referenced
this issue
Feb 14, 2024
# SEC: Dolibarr#28168 Correct protocol limitations (PHP7.4/Win) Protocol limitation was not active during test on windows platform. Moving the application of the limitation just before the curl_exec instruction made the limitation effective. Also extended the code to enable allowing ftp and ftps and extended the code for [CURLOPT_REDIR_PROTOCOLS_STR](https://www.php.net/manual/en/curl.constants.php#constant.curlopt-redir-protocols-str).
mdeweerd
added a commit
to mdeweerd/dolibarr
that referenced
this issue
Feb 14, 2024
# SEC: Dolibarr#28168 Correct protocol limitations (PHP7.4/Win) Protocol limitation was not active during test on windows platform. Moving the application of the limitation just before the curl_exec instruction made the limitation effective. Also extended the code to enable allowing ftp and ftps and extended the code for [CURLOPT_REDIR_PROTOCOLS_STR](https://www.php.net/manual/en/curl.constants.php#constant.curlopt-redir-protocols-str).
mdeweerd
added a commit
to mdeweerd/dolibarr
that referenced
this issue
Feb 14, 2024
# SEC: Dolibarr#28168 Correct protocol limitations (PHP7.4/Win) Protocol limitation was not active during test on windows platform. Moving the application of the limitation just before the curl_exec instruction made the limitation effective. Also extended the code to enable allowing ftp and ftps and extended the code for [CURLOPT_REDIR_PROTOCOLS_STR](https://www.php.net/manual/en/curl.constants.php#constant.curlopt-redir-protocols-str).
mdeweerd
added a commit
to mdeweerd/dolibarr
that referenced
this issue
Feb 14, 2024
# SEC: Dolibarr#28168 Correct protocol limitations (PHP7.4/Win) Protocol limitation was not active during test on windows platform. Moving the application of the limitation just before the curl_exec instruction made the limitation effective. Also extended the code to enable allowing ftp and ftps and extended the code for [CURLOPT_REDIR_PROTOCOLS_STR](https://www.php.net/manual/en/curl.constants.php#constant.curlopt-redir-protocols-str).
eldy
pushed a commit
that referenced
this issue
Feb 15, 2024
# SEC: #28168 Correct protocol limitations (PHP7.4/Win) Protocol limitation was not active during test on windows platform. Moving the application of the limitation just before the curl_exec instruction made the limitation effective. Also extended the code to enable allowing ftp and ftps and extended the code for [CURLOPT_REDIR_PROTOCOLS_STR](https://www.php.net/manual/en/curl.constants.php#constant.curlopt-redir-protocols-str).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bug
The test case wants to validate the an ftp request is not possible, however it gets a "Connection refused back" with a port number.
To make sure that an attempt to connect was made, I added a valid ftp server and got the following reply:
The reason this is possible is not obvious from the code.
Environment Version
develop/8592f6ba492/ed1e818ada9
Environment OS
Windows 10
Environment PHP
c:\wamp64\bin\php\php7.4.33
phpunit 9.5
Steps to reproduce the behavior
Run SecurityTest with phpunit (on windows).
The text was updated successfully, but these errors were encountered: