Skip to content

Comments

feat: delta metrics#866

Merged
fenos merged 1 commit intomasterfrom
observability/delta-metrics
Feb 20, 2026
Merged

feat: delta metrics#866
fenos merged 1 commit intomasterfrom
observability/delta-metrics

Conversation

@fenos
Copy link
Contributor

@fenos fenos commented Feb 20, 2026

What kind of change does this PR introduce?

Feat

What is the new behavior?

  • Support delta metrics
  • Fix vulnerability reported in Fastify

Additional context

Add any other context or screenshots.

@fenos fenos requested a review from a team as a code owner February 20, 2026 11:01
@coderabbitai
Copy link

coderabbitai bot commented Feb 20, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Summary by CodeRabbit

Release Notes

  • Bug Fixes

    • Requests with malformed Content-Type headers containing tab characters are now rejected with a 400 error response.
  • New Features

    • Added OTEL_METRICS_TEMPORALITY environment variable to configure metrics export temporality (DELTA or CUMULATIVE, defaults to CUMULATIVE).

Walkthrough

This pull request adds an early onRequest hook that checks the Content-Type header and responds with 400 if the header contains a tab character. It also introduces a new configuration field otelMetricsTemporality (values 'DELTA' | 'CUMULATIVE') read from OTEL_METRICS_TEMPORALITY (default 'CUMULATIVE') and used to set OTLP metric exporter temporality.

Sequence Diagram(s)


Comment @coderabbitai help to get the list of available commands and usage tips.

@fenos fenos force-pushed the observability/delta-metrics branch from 2378b2c to e9b6792 Compare February 20, 2026 11:34
@fenos fenos force-pushed the observability/delta-metrics branch from e9b6792 to 8719cc6 Compare February 20, 2026 11:36
@fenos fenos merged commit 2a4fa0a into master Feb 20, 2026
2 checks passed
@fenos fenos deleted the observability/delta-metrics branch February 20, 2026 11:44
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