Skip to content

Commit

Permalink
Added comments for HTTP response building code path (#10657)
Browse files Browse the repository at this point in the history
* Added CodeQL suppression justification comments for oop layer response handling code path.

* Updated codeql comment based on PR feedback.
  • Loading branch information
kshyju authored Dec 9, 2024
1 parent d8bfc69 commit ba3e9ef
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/WebJobs.Script/Binding/Http/RawScriptResult.cs
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,12 @@ public async Task ExecuteResultAsync(ActionContext context)
{
if (cookie.Item3 != null)
{
// CodeQL [SM02373] This code path constructs the cookie collection based on what the out-of-process function app (where customers can set these cookies) sends to the host. Overriding this behavior would introduce a breaking change for those customers.
response.Cookies.Append(cookie.Item1, cookie.Item2, cookie.Item3);
}
else
{
// CodeQL [SM02373] This code path constructs the cookie collection based on what the out-of-process function app (where customers can set these cookies) sends to the host. Overriding this behavior would introduce a breaking change for those customers.
response.Cookies.Append(cookie.Item1, cookie.Item2);
}
}
Expand Down

0 comments on commit ba3e9ef

Please sign in to comment.