Azure-Samples/hello-spring-function-azure

Use FunctionInvoker instead of AzureSpringBootRequestHandler in the sample

konjp opened this issue · 4 comments

konjp commented

org.springframework.cloud.function.adapter.azure.AzureSpringBootRequestHandler is deprecated in favor of org.springframework.cloud.function.adapter.azure.FunctionInvoker.

jdubois commented

Indeed, the API has evolved, but in my experience that new API is a bit buggy (I have NPE when executing my function as it's not correctly registered in the functions catalog, and also Spring Webflux doesn't seem well supported). This is very new, and it's not yet in the official documentation, so I suggest we wait a bit until we migrate to it.
If you have another experience, and have a working example, I'm happy to discuss this with you.
One of the good news is that Oleg told me he took some of my code and put it inside Spring Functions, so that should simplify some code here.

konjp commented

Maybe you can check this issue, and when 3.1.3 is released, then probably you can update sample. And it is in official documentation :-).

jdubois commented

Oh you're good @konjp - that's exactly the core of my issue!!! Oleg says it will be released tomorrow, so let's wait a bit, and let's do this.

  • 👍1
jdubois commented

This is fixed, thanks a lot @konjp !