Phu Dinh comments

Results 12 comments of


                                            Phu Dinh

Clarify normalization of methods taking paths

@markt-asf @gregw

Clarify normalization of methods taking paths

I reopened this issue for clarification of `ServletRequest#getRequestDispatcher(java.lang.String)`

Clarify on closing response before returning from dispatch forward()

Additional information from spec and doc: ``` Chapter 9. Dispatching Requests When building a web application, it is often useful to **forward processing of a request to another servlet**, or...

Clarify on closing response before returning from dispatch forward()

Can we clarify this in https://github.com/eclipse-ee4j/servlet-api/blob/master/spec/src/main/asciidoc/servlet-spec-body.adoc ? @stuartwdouglas @markt-asf @arjantijms

Clarify on closing response before returning from dispatch forward()

Check back on this clarification - what is response object (base vs wrapped) that should be closed in the forward case ?

Clarify on closing response before returning from dispatch forward()

@markt-asf Can you please give this one last review and see if it can make into Servlet-6.1 or close it out. Thanks!

Clarify servlet response sendRedirect()

yes, since all the browsers can support the relative URL in the Location header. If it is allowed, what should be the default? or it is up to the server's...

HttpServletResponse#sendRedirect(String) could use some improvement; needs 301/302/303 support; needs way to set request body

Hi @markt-asf For sendRedirect(String, int) or sendRedirect(String, int, boolean), should the API or doc specify what to do if the user use arbitrary status code other than 301, 302, or...

feat: Add content to the Jakarta Servlet Spec page

https://deploy-preview-684--jakartaee-specifications.netlify.app/specifications/servlet/6.1/ ``` Minimum Java SE Version Java SE 11 or higher ``` Should it be Java 17 or higher?

TCK: setMaxAgeZeroTest() expects different Max-Age between EE10 and EE11

@markt-asf