Bill Barker wrote:
<[EMAIL PROTECTED]> wrote in message news:[EMAIL PROTECTED]
luehe 2004/10/27 15:58:17
+
+ private Method[] getAllDeclaredMethods(Class c) {
+
+ if (c.equals(javax.servlet.http.HttpServlet.class)) {
+ return null;
+ }
+
+ Method[] parentMethods = getAllDeclaredMethods(c.getSuperclass());
If the servlet isn't a HttpServlet (e.g. it's a JSP page) then this will recurse down to j.l.Object, when c.getSuperClass will return 'null', and you will get an NPE from the 'c.equals' line.
IMHO, this patch is an overly complex way to try and determine something that isn't determinable under the servlet spec (again, think JSP page :). You might as well just set the Allow header to any old constant set of methods.
I agree being lazy has its good points: "Allow: GET, HEAD, POST" would be good enough for me :)
I don't like cut & pasting so much code :(
Rémy
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
