Skip to content

Add Content-Type to Request Header in Task SDK calls#57377

Merged
potiuk merged 1 commit intoapache:mainfrom
jscheffl:bugfix/add-content-type-to-task-sdk-calls
Oct 27, 2025
Merged

Add Content-Type to Request Header in Task SDK calls#57377
potiuk merged 1 commit intoapache:mainfrom
jscheffl:bugfix/add-content-type-to-task-sdk-calls

Conversation

@jscheffl
Copy link
Copy Markdown
Contributor

We deployed Airflow3 today first time together with Edge Worker in our environment. (I know way too late, but still now getting started...)

We received HTTP 403 from the client as our WAF rejected calls with POST/PATH w/o appropriate content-type sent with the request.

Whereas technically not needed and we know how communication runs between client and server, we needed to add an exception to make remote deployment via Edge possible.

Therefore I'd suggest - even if technically not needed - to add a proper content-type to request header if a body is sent This will lower trouble in other deployments hard to debug on WAF and especially in security hardened environment.

@jscheffl jscheffl added this to the Airflow 3.1.2 milestone Oct 27, 2025
@jscheffl jscheffl requested a review from ashb as a code owner October 27, 2025 20:44
@jscheffl jscheffl added the type:bug-fix Changelog: Bug Fixes label Oct 27, 2025
@potiuk potiuk merged commit 8695fc5 into apache:main Oct 27, 2025
85 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 27, 2025
)

(cherry picked from commit 8695fc5)

Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Oct 27, 2025
…che#57377)

(cherry picked from commit 8695fc5)

Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
potiuk pushed a commit that referenced this pull request Oct 27, 2025
) (#57386)

(cherry picked from commit 8695fc5)

Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:task-sdk type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants