Skip to content

Gate ChatGPT user ID metrics to app surfaces#13136

Closed
ningyi-oai wants to merge 1 commit intomainfrom
codex/app-user-id-metrics-telemetry
Closed

Gate ChatGPT user ID metrics to app surfaces#13136
ningyi-oai wants to merge 1 commit intomainfrom
codex/app-user-id-metrics-telemetry

Conversation

@ningyi-oai
Copy link

@ningyi-oai ningyi-oai commented Feb 28, 2026

Summary

  • add a ChatGPT user ID accessor in auth and thread it into OTLP metrics metadata
  • gate user ID emission behind an app-only opt-in env and a narrow originator allowlist so standalone OSS CLI stays unchanged
  • emit both enduser.id and user_id metric tags and add a focused metrics test

Tests

  • cargo fmt --manifest-path codex-rs/Cargo.toml --all
  • cargo test --manifest-path codex-rs/Cargo.toml -p codex-otel manager_attaches_chatgpt_user_id_tags_when_present (blocked in this fresh clone by git dependency fetch/setup)
  • cargo test --manifest-path codex-rs/Cargo.toml -p codex-core app_originator_allowlist_for_chatgpt_user_id_metrics_is_narrow (blocked in this fresh clone by git dependency fetch/setup)

Notes

  • Paired with openai/openai#726938 for the app-side opt-in env plumbing.

@github-actions
Copy link
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oai PRs contributed by OpenAI employees

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants