Graylog2 / Graylog2/collector

Remote config loading code could be more clear

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
triaged
Dominant language
Go
Stars
1
Forks
0
Avg merge
3d 1h
Merged PRs (30d)
7

Description

Small code quality issue, the code is currently a bit harder to follow than feels necessary:
https://github.com/Graylog2/collector/blob/044b82868894e0dea5408d976cd06a1612be9502/superv/configmanager/manager.go#L276-L307

Contributor guide

Open the contributing guide

Research direction

Start by reading superv/configmanager/manager.go at lines 276-307, the only location identified in the issue, and trace the remote config loading flow around it. Clarify the control flow without changing its behavior, then verify that remote config loading still works as before.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.