Skip to content

Add hard-coded Version.cs for telemetry - #75

Merged
kailash-b merged 1 commit into
masterfrom
chore/release-readiness
Jun 29, 2026
Merged

Add hard-coded Version.cs for telemetry#75
kailash-b merged 1 commit into
masterfrom
chore/release-readiness

Conversation

@kailash-b

@kailash-b kailash-b commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

✏️ Changes

This PR makes the SDK version reported in the Auth0-Client telemetry header explicit and release-managed, instead of deriving it from assembly reflection.

The version is now sourced from a hard-coded constant that release automation bumps on every release, matching

Added

  • Version.csinternal static class Version exposing Current ("1.0.0-beta.6"), the single constant used for telemetry.

Modified

  • Utils.CreateAgentString() — now builds the agent string from Version.Current instead of assembly reflection. Removed the System.Reflection import and the stale doc comment.
  • UtilsTests.cs — the name/version assertion now checks against Version.Current exactly rather than a loose regex.

Removed

  • Utils.BuildVersionString(Version?) — no longer needed once reflection is gone.
  • The BuildVersionString and assembly-null-fallback unit tests that covered it.

🔗 References

🎯 Testing

  • Full unit suite passes locally (610 passed, 0 failed).

  • This change adds unit test coverage

  • This change adds integration test coverage - N/A, telemetry header content is covered by unit tests.

  • This change has been tested on the latest version of the platform/language or why not

✅ Checklist

@kailash-b
kailash-b requested a review from a team as a code owner June 29, 2026 09:26
@codecov-commenter

codecov-commenter commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.50%. Comparing base (61ffe19) to head (64f751a).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   92.53%   92.50%   -0.04%     
==========================================
  Files          29       29              
  Lines        1433     1427       -6     
  Branches      164      163       -1     
==========================================
- Hits         1326     1320       -6     
  Misses         76       76              
  Partials       31       31              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kailash-b
kailash-b force-pushed the chore/release-readiness branch from 901af1a to 64f751a Compare June 29, 2026 09:29
@kailash-b
kailash-b merged commit 69cc4c0 into master Jun 29, 2026
3 checks passed
@kailash-b
kailash-b deleted the chore/release-readiness branch June 29, 2026 10:09
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.

3 participants