Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logging to get information about refresh token version #602

Merged
merged 6 commits into from
Jun 11, 2024

Conversation

cYKatherine
Copy link
Contributor

No description provided.

@cYKatherine cYKatherine force-pushed the kcc-add-metrics-to-track-v2-refresh-token-usage branch 2 times, most recently from 72e24de to 371a3ae Compare June 11, 2024 03:18
@cYKatherine cYKatherine self-assigned this Jun 11, 2024
@cYKatherine cYKatherine force-pushed the kcc-add-metrics-to-track-v2-refresh-token-usage branch 8 times, most recently from 0ba99f9 to 544e83c Compare June 11, 2024 05:13
@cYKatherine cYKatherine force-pushed the kcc-add-metrics-to-track-v2-refresh-token-usage branch from 544e83c to 149a399 Compare June 11, 2024 05:43
@cYKatherine cYKatherine force-pushed the kcc-add-metrics-to-track-v2-refresh-token-usage branch 3 times, most recently from 98680d8 to 154ba16 Compare June 11, 2024 06:16
@cYKatherine cYKatherine force-pushed the kcc-add-metrics-to-track-v2-refresh-token-usage branch from 154ba16 to e8a14b8 Compare June 11, 2024 06:24
@cYKatherine cYKatherine merged commit 6017673 into main Jun 11, 2024
4 checks passed
@cYKatherine cYKatherine deleted the kcc-add-metrics-to-track-v2-refresh-token-usage branch June 11, 2024 06:54
cYKatherine added a commit that referenced this pull request Jun 12, 2024
* Add logging to get information about refresh token version

* Fix unit tests

* Record metrics using labels to separate versions

* Remove unnecessary import

* Remove global registry metrics in TokenEncodingTest

* Remove unnecessary variables
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