Skip to content

CXF-8746: Support force GET on redirections#977

Closed
ypiel-talend wants to merge 1 commit intoapache:3.6.x-fixesfrom
ypiel-talend:ypiel/force_get_on_302
Closed

CXF-8746: Support force GET on redirections#977
ypiel-talend wants to merge 1 commit intoapache:3.6.x-fixesfrom
ypiel-talend:ypiel/force_get_on_302

Conversation

@ypiel-talend
Copy link
Contributor

CXF-8746: Support force GET on 301 & 302 redirections for historical reasons as explained in https://www.rfc-editor.org/rfc/rfc7231.html section 6.4.2 & 6.4.3.

@reta
Copy link
Member

reta commented Sep 11, 2022

@ypiel-talend my apologies, I have difficulties to understand this change, the sections you are referring to note that:

Note: For historical reasons, a user agent MAY change the request
method from POST to GET for the subsequent request. If this
behavior is undesired, the 307 (Temporary Redirect) status code
can be used instead.

This is the behavior driven by user agent (client) and not the server, I am not sure why CXF needs change here. Also, seems like this is really outdated behavior (from notes on spec), why do we need to support it? What is the use case?

Thank you.

@gnodet
Copy link
Contributor

gnodet commented Mar 12, 2026

Superseded by #2952 with a cleaner, non-API-breaking approach. Instead of threading a boolean forceGET parameter through abstract method signatures across multiple transport implementations, the new PR uses a contextual property (http.redirect.force.GET) and sets Message.HTTP_REQUEST_METHOD on the outMessage before retransmit, requiring changes only in HTTPConduit.java.

@gnodet
Copy link
Contributor

gnodet commented Mar 12, 2026

Superseded by #2952 which implements this with a minimal, non-API-breaking approach using a contextual property.

@gnodet gnodet closed this Mar 12, 2026
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.

3 participants