-
-
Notifications
You must be signed in to change notification settings - Fork 80
fix(azure): prevent conflicting topic subscription names #788
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #788 +/- ##
=======================================
Coverage 48.83% 48.83%
=======================================
Files 28 28
Lines 3551 3551
=======================================
Hits 1734 1734
Misses 1698 1698
Partials 119 119
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
03f64fb
to
7d84229
Compare
c8e2c09
to
a0e3e80
Compare
🎉 This PR is included in version 1.25.4 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
When topic names are long they result in subscriptions names that must be truncated (to max 24 chars) to meet Azure's name restrictions. The truncation could result in two services subscribing to the same topic ending up with the same name for their subscriptions, causing a conflict for the Event Subscription resources.