Skip to content

Routing: Export HealthCheckSettings - #6680

Open
meanwhile131 wants to merge 1 commit into
XTLS:mainfrom
meanwhile131:exportHealthCheck
Open

Routing: Export HealthCheckSettings#6680
meanwhile131 wants to merge 1 commit into
XTLS:mainfrom
meanwhile131:exportHealthCheck

Conversation

@meanwhile131

Copy link
Copy Markdown

Currently, trying to build a BurstObservatory config from conf.BurstObservatoryConfig:

config.BurstObservatory = &conf.BurstObservatoryConfig{
	SubjectSelector: []string{"out"},
	HealthCheck: &conf.healthCheckSettings{},
}

Results in a compiler error:

1. name healthCheckSettings not exported by package conf [UnexportedName]

conf.BurstObservatoryConfig is public, but uses conf.healthCheckSettings (a private struct), which prevents creating a conf.Config with a burst observatory from an outside package.
Making healthCheckSettings public should allow building of a config with a burst observatory.

To allow building of config by external packages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants