Graylog2 / fpm-recipes

Graylog package build recipes
http://docs.graylog.org/en/latest/pages/installation/operating_system_packages.html
18 stars 18 forks source link

Declare integrations_scripts_dir regardless of presence of enterprise plugins #136

Open drewmiranda-gl opened 2 months ago

drewmiranda-gl commented 2 months ago

This is to address edge cases where server.conf may be missing this configuration: https://github.com/Graylog2/graylog-plugin-enterprise/issues/7616

Because there is no default value, it can lead to unexpected and unwanted (and undocumented) behavior. We do (wrongly) document server.conf having a default value when in actuality it doesn't, or its default value is conditional on server.conf being created from the enterprise install package.

I think an elegant solution to this is to just include this config for all installs such as graylog-server in addition to graylog-enterprise

integrations_scripts_dir = /usr/share/graylog-server/scripts
tellistone commented 1 month ago

For this one, just need to add that line of config to be included in server.conf by default