Skip to content

Conversation

@tlater-famedly
Copy link
Contributor

This feature is purely cosmetic, we've avoided using it for anything; As expected, a consistency bug was caused by it (likely due to bad handling of the historic status of this as an arbitrarily generated ID), though the bug happened upstream in Zitadel.

@tlater-famedly tlater-famedly requested a review from a team as a code owner December 16, 2025 06:29
@CLAassistant
Copy link

CLAassistant commented Dec 16, 2025

CLA assistant check
All committers have signed the CLA.

@tlater-famedly tlater-famedly force-pushed the tlater/remove-custom-uid branch from e96f67a to c860bac Compare December 16, 2025 07:42
mzaniolo
mzaniolo previously approved these changes Dec 16, 2025
Copy link
Contributor

@mzaniolo mzaniolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

This feature is purely cosmetic, we've avoided using it for anything;
As expected, a consistency bug was caused by it (likely due to bad
handling of the historic status of this as an arbitrarily generated
ID), though the bug happened upstream in Zitadel.
CI really wants this fixed suddenly.
@tlater-famedly tlater-famedly merged commit 03c9828 into main Dec 16, 2025
3 checks passed
@tlater-famedly tlater-famedly deleted the tlater/remove-custom-uid branch December 16, 2025 08:30
@codecov
Copy link

codecov bot commented Dec 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.80%. Comparing base (6cd668e) to head (03c9828).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #158      +/-   ##
==========================================
- Coverage   89.98%   89.80%   -0.18%     
==========================================
  Files          10       10              
  Lines        1627     1628       +1     
==========================================
- Hits         1464     1462       -2     
- Misses        163      166       +3     
Files with missing lines Coverage Δ
src/lib.rs 78.32% <ø> (-1.12%) ⬇️
src/zitadel.rs 86.47% <100.00%> (-0.41%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cd668e...03c9828. Read the comment docs.

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.

5 participants