UseAzureAppConfiguration and AzureAppConfigurationRefreshMiddleware break context execution about appconfiguration HOT 11 OPEN

Hi @fjbelizon, we got a similar issue about durable functions being broken a while back, but ExecutionContext.SuppressFlow() wasn't mentioned. In the discussion there as well as in the linked issue that's in the Azure Functions repo, it was thought that the problem was not specific to the Azure App Configuration middleware.

One helpful test would be to see if version 6.0.1 of the provider also causes this issue for you, since that was before we added the call to SuppressFlow(). If the issue goes away, I'll need to update that original issue. Thanks!

from appconfiguration.

@amerjusupovic yes, same problem. Version 6.0.1 also breaks the execution of Eternal Functions.

I attach the code sample for you to test: bug-azure-app-configuration-refresh-middleware.zip. I've put two ‘TODO’ tags for you to comment/uncomment the lines that make the execution work/fail.

I look forward to your review. Thanks in advance.

from appconfiguration.

Any news about this issue? Thanks in advance

from appconfiguration.

@jimmyca15 Yes I believe so, I know someone on the functions team believed it wasn't an issue with the .NET provider library at first, but from this it seems like it is something we introduced. I'll look into this more and see if we can fix this portion of the code while still solving the HttpContext issue that originally required this change.

from appconfiguration.

is there any news about this issue? or any alternative, thanks in advance!!

from appconfiguration.

Hi @amerjusupovic. Ok, I'm going to do the test

from appconfiguration.

@amerjusupovic

Given there was a repro with 6.0.1 do you believe it would be the same issue as here?

from appconfiguration.

Ok, thanks @amerjusupovic 👌

from appconfiguration.

This looks like a dupe of this issue. I don't think ExecutionContext.SuppressFlow has any involvement here. This supported by 6.0.1 reproing as well.

from appconfiguration.

@amerjusupovic if you can confirm this is a dupe of the aforementioned issue we should merge this issue into the existing one which the Functions team mentioned they need to solve on their end.

from appconfiguration.

Sorry for the confusion in my previous response, yes the repro in 6.0.1 suggests it's the same issue, and that was before the call to SuppressFlow was added. This should be merged with the existing one.

from appconfiguration.