Ashish Chhabria

Results 16 comments of Ashish Chhabria

Prefer to have async as the implicit/default contract, and sync as the explicit contract. @SeanFeldman - recommend not having this change done, to avoid barriers to adopting the newest library...

I see the value, but we will have to support ```c# public virtual Task BeforeMessageSend(Message message); public virtual Task AfterMessageReceive(Message message); ``` along with ```c# public virtual Task BeforeMessageSendAsync(Message message);...

@SeanFeldman - we are moving the underlying issue to the [Azure SDK for .NET] (https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/servicebus/Microsoft.Azure.ServiceBus) repo. Once we archive this repo, the PR will be read only and we can...

@marcinkaluza - 10 seconds is probably too high for a time skew - a few milliseconds (or a couple seconds may be ok). Have you raised a support ticket with...

@SeanFeldman - we are moving the underlying issue to the [Azure SDK for .NET] (https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/servicebus/Microsoft.Azure.ServiceBus) repo. Once we archive this repo, the PR will be read only and we can...

can you share a repro? we can investigate.

@nemenos - thanks for reporting this - acknowledge the problem on my end too. this is on the server side, so we're investigating.

@ryancole - That is an interesting ask. the reason we don't support is is to avoid having a connection for a queue client that doesn't do anything. This would unfortunately...

@petrformanek thanks for reporting this. We confirmed that this is a bug. We'll add this to the backlog, but alternatively feel free to submit a PR and we can review...

@ApolloCreek Both Message and SystemPropertiesCollection are data objects so interface would not be helpful here. I agree with you that the 'internal' setter is not the best idea and we...