Fix drupal cookie interceptor

12815
Reporter: trobertson
Assignee: mdoering
Type: Bug
Summary: Fix drupal cookie interceptor
Priority: Blocker
Resolution: Fixed
Status: Closed
Created: 2013-02-26 14:36:03.948
Updated: 2013-08-29 14:44:38.706
Resolved: 2013-02-26 15:23:23.307
        
Description: Staging is giving everyone NPE:

java.lang.NullPointerException
    org.gbif.portal.struts.DrupalSessionInterceptor.findDrupalCookie(DrupalSessionInterceptor.java:67)]]>
    

Author: trobertson@gbif.org
Comment: https://code.google.com/p/gbif-portal/source/detail?r=1424 for now
Created: 2013-02-26 14:37:31.182
Updated: 2013-02-26 14:37:31.182

Author: mdoering@gbif.org
Comment: The way we use to get the HttpServletRequest in a struts2 interceptor via the ActionContext is fine and even the example given in the StrutsStatics java docs. It should also never be null, so really the null test should not be needed. Thanks for the cookie null test patch - I've overseen that and expected an empty array.
Created: 2013-02-26 15:11:37.494
Updated: 2013-02-26 15:11:37.494