-
Notifications
You must be signed in to change notification settings - Fork 2k
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
AAD authentication for Azure Event Hub #6667
Conversation
eventData.SetPartitionKey(this.StreamId.Guid.ToString()); | ||
var wrapper = new WrappedEventData(eventData.Body, eventData.Properties, eventData.SystemProperties, | ||
partitionKey: StreamId.Guid.ToString(), | ||
offset: DateTime.UtcNow.Ticks, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about this change. Previously offset was set to StreamId.Guid.ToString() + now.ToString
, but this is impossible now that it's long
.
There are some conflicts now that the other PR is merged. Could you please rebase on master and force-push? |
@aelij thank your for your PR! I am ok to merge it, if you could resolve the conflict. Otherwise I can resolve them myself, since they are most likely due to my I don't think we will backport this PR to 3.3.0 to limit the risk though. |
@benjaminpetit I'd appreciate it if you could resolve them |
Use Azure.Identity only in tests
/azp run Azure DevOps - Functional |
Azure Pipelines successfully started running 1 pipeline(s). |
Note there are a few minor breaking changes:
long
instead ofstring
EventData
is immutable, so I've removedEventDataProxyMethods