Update the sample to reflect the new DI integration approach. #39
Update the sample to reflect the new DI integration approach. #39tzolov wants to merge 2 commits into
Conversation
tzolov
commented
Jan 31, 2023
- New DI approach replaces the legacy FunctionInvoker and treats the Azure Function class as yet another Spring component, allowing auto-wiring of SCF or any other Spring services.
- The PR also adds hardcoded CORS configuration to make it easy to test in the azure platform.
- lastly the PR simplifies the pom configuration.
|
@tzolov please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
|
I believe we did the same work on the pom.xml, and I already merged mine :-) |
|
@tzolov I'm getting this error when running |
|
@jdubois , maybe your local Azure core tools are outdated? Here are mine: The DI integration relies on a new Azure Java Runtime feature. |
|
No I have everything automatically up to date: |
|
This is award because i can not reproduce the issue. The |
|
Indeed I have a difference: I just reinstalled everything with Homebrew and I still don't have the same file as you - I don't think this is OS-specific, and you seem to be using Homebrew like me, but just to check: I'm using a MacBook Pro M1, what are you using? |
|
After talking directly with @tzolov : he has an unpublished version of the worker, so we'll need to wait a few weeks before this is generally available. |