gandres / pwm

Automatically exported from code.google.com/p/pwm
0 stars 0 forks source link

Allow plus signs in profile email addresses #433

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. a user wants to update their email address
2. user goes to "update profile"
3. user tries to use a plus sign (+) in their email address

What is the expected output? What do you see instead?
Expected: plus signs are allowed
Actual: plus signs not allowed ... NPE (see below)

What version of PWM are you using?
Latest

What ldap directory and version are you using?
AD 2008 R2

Please paste any error log messages below:
2013-07-23 13:04:30, TRACE, pwm.SessionFilter, {bt} POST request for: 
/pwm/private/UpdateProfile  [10.150.214.5/basefdt14k01.snc.dev.lab]
  mail='rhammond+spam@fosgail.com'
  pwmFormID='fDxDFoU5ytkY5i3dblPOOb3Ul9qaX1Dq1400ba11013ly0v2f'
  processAction='updateProfile'
2013-07-23 13:04:32, INFO , servlet.UpdateProfileServlet, updating profile for 
CN=ADMIN-Ryan Hammond,OU=blah,blah,blah
2013-07-23 13:04:32, TRACE, provider.WireTraceWrapper, begin op#17 
getDirectoryVendor()
2013-07-23 13:04:32, TRACE, provider.WireTraceWrapper, finish op#17 result: 
MICROSOFT_ACTIVE_DIRECTORY (1ms)
2013-07-23 13:04:32, WARN , servlet.TopServlet, {bt} unexpected pwm error 
during page generation: null [10.150.214.5/basefdt14k01.snc.dev.lab]
java.lang.NullPointerException
        at password.pwm.util.Helper.writeFormValuesToLdap(Helper.java:662)
        at password.pwm.servlet.UpdateProfileServlet.doProfileUpdate(UpdateProfileServlet.java:244)
        at password.pwm.servlet.UpdateProfileServlet.advanceToNextStep(UpdateProfileServlet.java:143)
        at password.pwm.servlet.UpdateProfileServlet.processRequest(UpdateProfileServlet.java:102)
        at password.pwm.servlet.TopServlet.handleRequest(TopServlet.java:82)
        at password.pwm.servlet.TopServlet.doPost(TopServlet.java:144)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:728)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:305)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
        at password.pwm.AuthenticationFilter.processAuthenticatedSession(AuthenticationFilter.java:134)
        at password.pwm.AuthenticationFilter.doFilter(AuthenticationFilter.java:82)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:243)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
        at password.pwm.SessionFilter.processFilter(SessionFilter.java:254)
        at password.pwm.SessionFilter.doFilter(SessionFilter.java:128)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:243)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
        at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:222)
        at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:123)
        at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:171)
        at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:99)
        at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:953)
        at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:118)
        at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:408)
        at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1023)
        at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:589)
        at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:310)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:724)

Original issue reported on code.google.com by rhamm...@fosgail.com on 23 Jul 2013 at 1:33

GoogleCodeExporter commented 9 years ago
Fixed in revision 589.  Exception appears unrelated and does not correspond to 
current code base.  If you still getting exception please open a new issue.

Original comment by jrivard on 16 Aug 2013 at 8:40