groovy-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yazad Khambata <yaz...@gmail.com>
Subject GROOVY-5227 - AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet
Date Mon, 05 Jun 2017 04:53:44 GMT
Hi Paul,

I have analyzed the ticket "GROOVY-5227 - AbstractHttpServlet should
extend GenericServlet directly instead of HttpServlet" and provided
Detailed Analysis, and suggested solutions on the ticket.

In brief,
1. We should NOT extends GenericServlet as asked in the ticket.
2. We must either not override service directly (as asked in the
ticket) or document this behavior, since by doing so,
2.a. We make it difficult for someone using GroovyServlet to extend
the servlet and do something specific when say the script is invoked
with POST or PUT isntead of GET. But this has a not-so-elegant
workaround - conditional code in the script using the implicit request
object's getMethod method.
2.b. We loose getLastModified which sends back 304 Not Modified
response header for efficiency, if nothing has changed.

Please see detailed info here,
https://issues.apache.org/jira/browse/GROOVY-5227?focusedCommentId=16036524&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16036524

Regards,
Yazad Khambata

Mime
View raw message