Skip to content

extproc: sign content-length for AWS#788

Merged
mathetake merged 1 commit intomainfrom
extprocrevet
Jun 27, 2025
Merged

extproc: sign content-length for AWS#788
mathetake merged 1 commit intomainfrom
extprocrevet

Conversation

@mathetake
Copy link
Copy Markdown
Member

@mathetake mathetake commented Jun 27, 2025

Description

Previously, we did not sign the content-length header due to a quirkiness introduced in #721. Now that #721 was reverted in #730, the content-length is properly set, so we can sign the content-length.

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
@mathetake mathetake marked this pull request as ready for review June 27, 2025 17:53
@mathetake mathetake requested a review from a team as a code owner June 27, 2025 17:53
@mathetake mathetake merged commit 92e5202 into main Jun 27, 2025
26 checks passed
@mathetake mathetake deleted the extprocrevet branch June 27, 2025 18:01
yuzisun added a commit to yuzisun/ai-gateway that referenced this pull request Jul 2, 2025
This reverts commit 92e5202.

Signed-off-by: Dan Sun <dsun20@bloomberg.net>
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.

2 participants