[WebPubSub]Add isolated-process extension for WebPubSub

[JialinXin]

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

[azure-sdk]

API change check

APIView has identified API level changes in this PR and created following API reviews.

Microsoft.Azure.Functions.Worker.Extensions.WebPubSub

[jsquire]

@JoshLove-msft: Would you please review?

[JoshLove-msft]

Are we able to add tests?

[JialinXin]

Are we able to add tests?

Not much needs as these are simply type definitions. Refers to a few isolated-process extensions are usually very simple.

[JialinXin]

Thanks for sharing the context. From my E2E testing, single output is good to work though multiple array output will throw Object Null exceptions.

Is there more info in the stack trace that points to where this is happening?

Unfortunately no. Even --verbose doesn't show much details. And I'm not able to attach debug to it from neither host project nor extension project, which attaches the func.exe.

I have a sample repo if you'd like to have a try. https://github.com/JialinXin/FunctionSamples/tree/main/IsolatedProcess/SimpleOutput

[2023-03-13T05:32:29.147Z] Executing 'Functions.Notification' (Reason='This function was programmatically called via the host APIs.', Id=3c4c4ca8-9a51-4d46-932e-2f4e52b650a0)
[2023-03-13T05:32:29.160Z] C# HTTP trigger function processed a request.
[2023-03-13T05:32:29.194Z] Executed 'Functions.Notification' (Failed, Id=3c4c4ca8-9a51-4d46-932e-2f4e52b650a0, Duration=45ms)
[2023-03-13T05:32:29.195Z] System.Private.CoreLib: Exception while executing function: Functions.Notification. Azure.Messaging.WebPubSub: Value cannot be null. (Parameter 'hub').
[2023-03-13T05:32:29.198Z] Executed HTTP request: {
[2023-03-13T05:32:29.199Z]   requestId: "ba066216-6cdd-4a3d-88c4-8692fdf202b3",
[2023-03-13T05:32:29.201Z]   identities: "(WebJobsAuthLevel:Admin)",
[2023-03-13T05:32:29.202Z]   status: "500",
[2023-03-13T05:32:29.203Z]   duration: "62"
[2023-03-13T05:32:29.204Z] }

[JialinXin]

@JoshLove-msft Can we release an init version to support isolated-process Web PubSub regarding a few customer asks.

[JialinXin]

@JoshLove-msft Do you have any comments to move on review the PR? We get another few asks for isolated-process support on Web PubSub service. I believe this is due to current fact that .NET7.0 only supports isolated-process now.

[JialinXin]

@JoshLove-msft Appreciate the review! ❤️

I just merge with latest code to resolve the conflict and did some tests again to ensure it's working with latest. I notice a few more things about array output failure. Would you share some insights if I can improve that? Though it could be worked around by the multiple output solution. So I think it's OK if not able to support in the initial preview version if I couldn't find a way right now.

I think the problem might be it's not able to resolve the multiple different derived types under WebPubSubAction,
for example, if the return is SendToAllAction[] is good, but mixed WebPubSubAction[](SendToAllAction, SendToGroupAction...) is not. Is there some special handling in the isolated process I should take care?

[drguthrie]

Sending/receiving Binary data does not appear to work. If I send X bytes, ~X*1.33 bytes are received. I would guess there is a Base64 encoding at play….however I tried to decode the received data and it failed. I have worked around it by manually Base64 encoding before sending as text then manually decoding. Note this is not ideal as the sender is a resource restricted microcontroller.

[vicancy]

Hi @drguthrie, could you share more about how you "send X bytes" and how you read the received bytes?