Skip to content

Tweak telemetry collection#3316

Merged
xavdid-stripe merged 1 commit intomasterfrom
RUN_DEVSDK-2244
Mar 10, 2026
Merged

Tweak telemetry collection#3316
xavdid-stripe merged 1 commit intomasterfrom
RUN_DEVSDK-2244

Conversation

@xavdid-stripe
Copy link
Member

Why?

We've long collected detailed platform information about calls made using the SDK. But a user brought to our attention that we're probably collecting more than we need. To fix, we're no longer sending the full uname output and only collecting data we actually care about. Additionally, users can opt out of sending platform information using the existing telemetry opt-out methods.

What?

  • remove unused publisher User-Agent key
  • condense important data from uname into the platform key
  • tests

See Also

@xavdid-stripe xavdid-stripe requested a review from a team as a code owner March 10, 2026 01:09
@xavdid-stripe xavdid-stripe requested review from mbroshi-stripe and removed request for a team March 10, 2026 01:09
@xavdid-stripe xavdid-stripe enabled auto-merge (squash) March 10, 2026 01:09
@xavdid-stripe xavdid-stripe merged commit 786441f into master Mar 10, 2026
9 checks passed
@xavdid-stripe xavdid-stripe deleted the RUN_DEVSDK-2244 branch March 10, 2026 13:16
xavdid-stripe added a commit that referenced this pull request Mar 10, 2026
* add .claude directory (#3307)

Co-authored-by: Michael Broshi <94012587+mbroshi-stripe@users.noreply.github.com>

* Add agent information to UserAgent (#3310)

* Add Stripe-Request-Trigger header (#3313)

* add "prepare" to justfile

* add Stripe-Request-Trigger header

* revert env var change

* add new agent keys (#3315)

* revert justfile variable change

* Bump version to 50.4.1

* tweak telemetry collection (#3316)

---------

Co-authored-by: Michael Broshi <94012587+mbroshi-stripe@users.noreply.github.com>
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