YunoHost / package_linter

Linter for YunoHost applications packages
https://yunohost.org/#/packaging_apps
GNU Affero General Public License v3.0
17 stars 13 forks source link

Warn about the use of add_header in nginx conf #45

Closed alexAubin closed 5 years ago

alexAubin commented 5 years ago

c.f. https://www.peterbe.com/plog/be-very-careful-with-your-add_header-in-nginx and https://github.com/YunoHost-Apps/nextcloud_ynh/pull/160

Apps should not use add_header in nginx conf, but rather more_set_headers ...

This adds a warning in package_linter.

alexAubin commented 5 years ago

(Planning to merge this in ~3 days)