{"action":"created","issue":{"url":"https://api.github.com/repos/eclipse/jetty.project/issues/660","repository_url":"https://api.github.com/repos/eclipse/jetty.project","labels_url":"https://api.github.com/repos/eclipse/jetty.project/issues/660/labels{/name}","comments_url":"https://api.github.com/repos/eclipse/jetty.project/issues/660/comments","events_url":"https://api.github.com/repos/eclipse/jetty.project/issues/660/events","html_url":"https://github.com/eclipse/jetty.project/issues/660","id":161674342,"number":660,"title":"NullPointerException in Request.getParameter: _parameters is null","user":{"login":"oyvindhorneland","id":1670211,"avatar_url":"https://avatars.githubusercontent.com/u/1670211?v=3","gravatar_id":"","url":"https://api.github.com/users/oyvindhorneland","html_url":"https://github.com/oyvindhorneland","followers_url":"https://api.github.com/users/oyvindhorneland/followers","following_url":"https://api.github.com/users/oyvindhorneland/following{/other_user}","gists_url":"https://api.github.com/users/oyvindhorneland/gists{/gist_id}","starred_url":"https://api.github.com/users/oyvindhorneland/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oyvindhorneland/subscriptions","organizations_url":"https://api.github.com/users/oyvindhorneland/orgs","repos_url":"https://api.github.com/users/oyvindhorneland/repos","events_url":"https://api.github.com/users/oyvindhorneland/events{/privacy}","received_events_url":"https://api.github.com/users/oyvindhorneland/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/eclipse/jetty.project/labels/Bug","name":"Bug","color":"eb6420"}],"state":"open","locked":false,"assignee":{"login":"joakime","id":191721,"avatar_url":"https://avatars.githubusercontent.com/u/191721?v=3","gravatar_id":"","url":"https://api.github.com/users/joakime","html_url":"https://github.com/joakime","followers_url":"https://api.github.com/users/joakime/followers","following_url":"https://api.github.com/users/joakime/following{/other_user}","gists_url":"https://api.github.com/users/joakime/gists{/gist_id}","starred_url":"https://api.github.com/users/joakime/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joakime/subscriptions","organizations_url":"https://api.github.com/users/joakime/orgs","repos_url":"https://api.github.com/users/joakime/repos","events_url":"https://api.github.com/users/joakime/events{/privacy}","received_events_url":"https://api.github.com/users/joakime/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/eclipse/jetty.project/milestones/2","html_url":"https://github.com/eclipse/jetty.project/milestones/9.3.x","labels_url":"https://api.github.com/repos/eclipse/jetty.project/milestones/2/labels","id":1583582,"number":2,"title":"9.3.x","description":"Issue should be targeted for resolution in 9.3.x releases.","creator":{"login":"jmcc0nn3ll","id":273327,"avatar_url":"https://avatars.githubusercontent.com/u/273327?v=3","gravatar_id":"","url":"https://api.github.com/users/jmcc0nn3ll","html_url":"https://github.com/jmcc0nn3ll","followers_url":"https://api.github.com/users/jmcc0nn3ll/followers","following_url":"https://api.github.com/users/jmcc0nn3ll/following{/other_user}","gists_url":"https://api.github.com/users/jmcc0nn3ll/gists{/gist_id}","starred_url":"https://api.github.com/users/jmcc0nn3ll/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmcc0nn3ll/subscriptions","organizations_url":"https://api.github.com/users/jmcc0nn3ll/orgs","repos_url":"https://api.github.com/users/jmcc0nn3ll/repos","events_url":"https://api.github.com/users/jmcc0nn3ll/events{/privacy}","received_events_url":"https://api.github.com/users/jmcc0nn3ll/received_events","type":"User","site_admin":false},"open_issues":5,"closed_issues":33,"state":"open","created_at":"2016-02-15T15:30:50Z","updated_at":"2016-06-22T17:44:37Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2016-06-22T12:55:36Z","updated_at":"2016-06-23T02:21:03Z","closed_at":null,"body":"**Version**: 9.3.9.v20160517 and 9.3.10.v20160621\r\n\r\n**Problem**: Encountering a NullPointerException when the first servlet filter tries to invoke request.getParameter for a POST request. \r\n\r\nDebug in Request class shows that \r\n\r\n- `_parameters` is null in Request.java:961. \r\n- `_queryParameters` is an empty list\r\n- `_contentParameters` is null\r\n\r\nSince `_queryParameters` is an empty list, `restoreParameters()` assigns `_parameters=_contentParameters`, which is in this case is null. If I manually in debug mode execute `extractContentParameters()`, `_contentParameters` is populated and the NullPointer is then avoided.\r\n\r\n**Stacktrace**:\r\n`java.lang.NullPointerException\r\n\tat org.eclipse.jetty.server.Request.getParameter(Request.java:961)\r\n\tat com.iknowbase.logging.http.RequestEntryFilter.getConversationId(RequestEntryFilter.java:131)\r\n\tat com.iknowbase.logging.http.RequestEntryFilter.doFilterInternal(RequestEntryFilter.java:70)\r\n\tat org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:107)\r\n\tat org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1676)\r\n\tat org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:581)\r\n\tat org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:143)\r\n\tat org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:548)\r\n\tat org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:226)\r\n\tat org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1180)\r\n\tat org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:511)\r\n\tat org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:185)\r\n\tat org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1112)\r\n\tat org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)\r\n\tat org.eclipse.jetty.server.handler.gzip.GzipHandler.handle(GzipHandler.java:397)\r\n\tat org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:119)\r\n\tat org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:134)\r\n\tat org.eclipse.jetty.server.Server.handle(Server.java:524)\r\n\tat org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:319)\r\n\tat org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:253)\r\n\tat org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:273)\r\n\tat org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:95)\r\n\tat org.eclipse.jetty.io.ssl.SslConnection.onFillable(SslConnection.java:201)\r\n\tat org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:273)\r\n\tat org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:95)\r\n\tat org.eclipse.jetty.io.SelectChannelEndPoint$2.run(SelectChannelEndPoint.java:93)\r\n\tat org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.executeProduceConsume(ExecuteProduceConsume.java:303)\r\n\tat org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.produceConsume(ExecuteProduceConsume.java:148)\r\n\tat org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.run(ExecuteProduceConsume.java:136)\r\n\tat org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:671)\r\n\tat org.eclipse.jetty.util.thread.QueuedThreadPool$2.run(QueuedThreadPool.java:589)\r\n\tat java.lang.Thread.run(Thread.java:745)`\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/eclipse/jetty.project/issues/comments/227932430","html_url":"https://github.com/eclipse/jetty.project/issues/660#issuecomment-227932430","issue_url":"https://api.github.com/repos/eclipse/jetty.project/issues/660","id":227932430,"user":{"login":"gregw","id":70829,"avatar_url":"https://avatars.githubusercontent.com/u/70829?v=3","gravatar_id":"","url":"https://api.github.com/users/gregw","html_url":"https://github.com/gregw","followers_url":"https://api.github.com/users/gregw/followers","following_url":"https://api.github.com/users/gregw/following{/other_user}","gists_url":"https://api.github.com/users/gregw/gists{/gist_id}","starred_url":"https://api.github.com/users/gregw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gregw/subscriptions","organizations_url":"https://api.github.com/users/gregw/orgs","repos_url":"https://api.github.com/users/gregw/repos","events_url":"https://api.github.com/users/gregw/events{/privacy}","received_events_url":"https://api.github.com/users/gregw/received_events","type":"User","site_admin":false},"created_at":"2016-06-23T02:21:02Z","updated_at":"2016-06-23T02:21:02Z","body":"@joakime I think this fix is wrong.    The NO_PARAMS is a special value above and beyond just having no values in it.   We need to retore the parameters as they were before, not create a new map that just happens to have the same content."}}
{"actor":{"display_login":"gregw"}}