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
Adds possibility to configure protocols for allowed outgoing connections #16910
Conversation
config/global.ini.php
Outdated
@@ -719,6 +719,10 @@ | |||
; If set to 0 it also disables the "sent plugin update emails" feature in general and the related setting in the UI. | |||
enable_update_communication = 1 | |||
|
|||
; This option defines the protocols Matomo's Http class is allowed to open. | |||
; If you may need to download GeoIP updates or other stuff using other protocols like ftp you may need to extend this list. | |||
allowed_protocols = 'http,https' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think it might be better to name this allowed_outgoing_protocols
or similar? To avoid confusion about what it refers to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated the name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a comment, generally looks good, think the UI tests will have to be updated
@sgiehl seems like there are some test failures |
@diosmosis should be fixed now |
Description:
Matomo should normally not send any request using other protocols than http and https.
In some uncommon use cases it might be needed to send other request. This can be done using a config setting to define the list of allowed protocols.
Review