Skip to content

Send env reload response with "Failure" as Status property value when app payload is not present#1933

Merged
kshyju merged 1 commit intomainfrom
shkr/1931_functionsnethost_fail_specialization-when-no-payload
Oct 4, 2023
Merged

Send env reload response with "Failure" as Status property value when app payload is not present#1933
kshyju merged 1 commit intomainfrom
shkr/1931_functionsnethost_fail_specialization-when-no-payload

Conversation

@kshyju
Copy link
Copy Markdown
Member

@kshyju kshyju commented Sep 26, 2023

Fixes #1931

Currently, If we get specialization request without a valid payload present in the FunctionAppDirectory , our code crashes when trying to load the assembly. This PR updates that code path so that it will send an instance of EnvironmentReloadResponse with "Failure" as the "Status" property value.

@kshyju kshyju requested review from brettsam and fabiocav September 26, 2023 18:53
@kshyju
Copy link
Copy Markdown
Member Author

kshyju commented Sep 26, 2023

@paulbatum @TsuyoshiUshio FYI

Comment thread host/src/FunctionsNetHost/Grpc/IncomingGrpcMessageHandler.cs Outdated
… specialization request received and no payload is present.
@kshyju kshyju force-pushed the shkr/1931_functionsnethost_fail_specialization-when-no-payload branch from 16b36ff to 1504dee Compare September 28, 2023 23:19
@kshyju kshyju merged commit dfc1801 into main Oct 4, 2023
@kshyju kshyju deleted the shkr/1931_functionsnethost_fail_specialization-when-no-payload branch October 4, 2023 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle specialization without app payload

3 participants