Skip to content

fix: Add parameter validation for config#949

Merged
burningalchemist merged 1 commit intoburningalchemist:masterfrom
destinyoooo:fix_config
Mar 26, 2026
Merged

fix: Add parameter validation for config#949
burningalchemist merged 1 commit intoburningalchemist:masterfrom
destinyoooo:fix_config

Conversation

@destinyoooo
Copy link
Contributor

@destinyoooo destinyoooo commented Mar 26, 2026

This PR includes the following improvements:

  1. Add parameter validation for config ：
  • Enhanced checkRequiredFields method in config/config.go
  • Validated target configuration to ensure data_source_name and collectors fields exist
  • Validated jobs configuration to ensure job_name, collectors, static_configs, and targets fields exist

@destinyoooo destinyoooo mentioned this pull request Mar 26, 2026
Closed
burningalchemist
burningalchemist approved these changes Mar 26, 2026
View reviewed changes
Copy link
Owner

@burningalchemist burningalchemist left a comment

Choose a reason for hiding this comment

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

🚀

@burningalchemist burningalchemist merged commit 43e3fbf into burningalchemist:master Mar 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@burningalchemist burningalchemist burningalchemist approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@destinyoooo @burningalchemist