Skip to content
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

Add allowed_auxiliary_paths to default server.conf file #10951

Merged
merged 1 commit into from
Jul 1, 2021
Merged

Conversation

danotorrey
Copy link
Contributor

@danotorrey danotorrey commented Jun 29, 2021

Description

Add a sample configuration for the allowed_auxiliary_paths configuration property in the default graylog.conf file, which was originally added in #10244

Documentation has also been provided in Graylog2/documentation#1185.

Motivation and Context

This configuration property was discussed yesterday in the context of investigating a Cloud-related issue. I thought that it would be good to provide a sample configuration and documentation for this property, so that others can see a description for the property and an example for how to configure it.

How Has This Been Tested?

No testing needed. Just static file changes in the default graylog.conf, which are commented out.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [*] My code follows the code style of this project.
  • [*] My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

@mpfz0r mpfz0r merged commit 8d259c4 into master Jul 1, 2021
@mpfz0r mpfz0r deleted the aux-path-conf branch July 1, 2021 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants