Add ingestion key check to otel collector via OpAMP #825
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| allowed_headers: ['*'] | ||
| include_metadata: true | ||
| endpoint: '0.0.0.0:4318' | ||
| # otlp/hyperdx: |
There was a problem hiding this comment.
moved to opampController, this is so we don't enable these receivers until we know for sure which authentication mode we want
| @@ -57,7 +62,9 @@ services: | |||
| - internal | |||
| healthcheck: | |||
| # "clickhouse", "client", "-u ${CLICKHOUSE_USER}", "--password ${CLICKHOUSE_PASSWORD}", "-q 'SELECT 1'" | |||
| test: wget --no-verbose --tries=1 http://127.0.0.1:8123/ping || exit 1 | |||
| test: | |||
| wget -O /dev/null --no-verbose --tries=1 http://127.0.0.1:8123/ping || | |||
There was a problem hiding this comment.
fixes failing healthcheck (dev issue)
| @@ -32,8 +33,10 @@ processors: | |||
| statements: | |||
| # JSON parsing: Extends log attributes with the fields from structured log body content, either as an OTEL map or | |||
| # as a string containing JSON content. | |||
| - set(log.cache, ExtractPatterns(log.body, "(?P<0>(\\{.*\\}))")) where IsString(log.body) | |||
| - merge_maps(log.attributes, ParseJSON(log.cache["0"]), "upsert") where IsMap(log.cache) | |||
| - set(log.cache, ExtractPatterns(log.body, "(?P<0>(\\{.*\\}))")) where | |||
There was a problem hiding this comment.
just prettier formatting, no changes here in transform
| if (config.IS_LOCAL_APP_MODE) { | ||
| return LOCAL_APP_TEAM; | ||
| } | ||
|
|
||
| return Team.findById(id, fields); | ||
| return Team.findOne({}, fields); |
There was a problem hiding this comment.
we should always only have 1 team in the collection
| @@ -9,6 +9,7 @@ export interface ITeam { | |||
| allowedAuthMethods?: 'password'[]; | |||
| apiKey: string; | |||
| hookId: string; | |||
| collectorAuthenticationEnforced: boolean; | |||
There was a problem hiding this comment.
this enables collector auth moving forward, but preserves existing v2 no-auth behavior for existing deployments
HDX-1698
TODO: