-
Notifications
You must be signed in to change notification settings - Fork 457
Fix invocation timeout when incoming request contains "x-ms-invocation-id" header #10980
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test/WebJobs.Script.Tests/HttpProxyService/DefaultHttpProxyServiceTests.cs
Outdated
Show resolved
Hide resolved
…github.com/Azure/azure-functions-host into satvu/fix-invocation-timeout-due-to-header
I am struggling to recall this exactly. But my guess would be something along the lines of putting the existing header into a new header, like |
…github.com/Azure/azure-functions-host into satvu/fix-invocation-timeout-due-to-header
…github.com/Azure/azure-functions-host into satvu/fix-invocation-timeout-due-to-header
I'm OK with us scoping the changes to what is currently in place. If we feel there's a need for us to maintain the original header (I don't think we've been able to identify valid scenarios for that), we can introduce that in another PR. That would also lead to questions about hops across multiple Function Apps. |
Issue describing the changes in this PR
resolves #10934
Pull request checklist
IMPORTANT: Currently, changes must be backported to the
in-proc
branch to be included in Core Tools and non-Flex deployments.in-proc
branch is not requiredrelease_notes.md