Skip to content

Support custom server config prefix#26524

Closed
xd4rker wants to merge 5 commits into
masterfrom
feature/support-custom-config-prefix
Closed

Support custom server config prefix#26524
xd4rker wants to merge 5 commits into
masterfrom
feature/support-custom-config-prefix

Conversation

@xd4rker

@xd4rker xd4rker commented Jun 30, 2026

Copy link
Copy Markdown
Member

Description

Introduced GRAYLOG_CONFIG_PREFIX, which allows overriding the prefix used to read server configuration from environment variables and system properties.

How Has This Been Tested?

See unit tests.

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 requested a documentation update.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

@xd4rker xd4rker requested a review from a team June 30, 2026 13:14
Comment thread graylog2-server/src/main/java/org/graylog2/ConfigurationPrefixes.java Outdated
@xd4rker xd4rker marked this pull request as ready for review June 30, 2026 14:33

@todvora todvora left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure how to deal with the hardcoded GRAYLOG_CONFIG_PREFIX property for the customization.

@todvora

todvora commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Closing, this will be solved elsewhere, differently.

@todvora todvora closed this Jun 30, 2026
@xd4rker xd4rker deleted the feature/support-custom-config-prefix branch July 1, 2026 07:31
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