[
https://issues.apache.org/jira/browse/WICKET-7152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17938847#comment-17938847
]
ASF GitHub Bot commented on WICKET-7152:
----------------------------------------
hosea commented on code in PR #1133:
URL: https://github.com/apache/wicket/pull/1133#discussion_r2015974772
##########
wicket-request/src/main/java/org/apache/wicket/request/http/handler/RedirectRequestHandler.java:
##########
@@ -142,7 +148,7 @@ else if (mode == Mode.STATUS)
setStatus(response, requestCycle, location);
}
// Mode.AUTO
- else if (status == HttpServletResponse.SC_MOVED_TEMPORARILY)
+ else if (status == HttpServletResponse.SC_FOUND)
Review Comment:
Hi, SC_FOUND is the same code as SC_MOVED_TEMPORARILY (302). There is a
comment in the source file of HttpServletResponse concerning
SC_MOVED_TEMPORARILY: "SC_FOUND is now the preferred definition." That's the
reason why I changed it.
> Extend RedirectRequestHandler for statusCodes 307 and 308
> ---------------------------------------------------------
>
> Key: WICKET-7152
> URL: https://issues.apache.org/jira/browse/WICKET-7152
> Project: Wicket
> Issue Type: New Feature
> Components: wicket-core
> Affects Versions: 10.4.0
> Reporter: Hans Schäfer
> Priority: Minor
>
> RedirectRequestHandler only accepts status 301, 302 and 303. Should also work
> for other status codes 307 and 308.
> 308 makes the browser use the same HTTP-Method. So if a browser sends a
> POST-Request and receives a 308-Response, then the browser will repost for
> the changed location. I need this behavior for supporting an endpoint used by
> another company. My workaround until now is an Apache-Rule doing the job now.
> Note: If it was a 302 the browser would change the method to "GET".
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
