Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Servlet 4 get/setInitParameter() and get/setAttribute() must throw NPE if arg is null #4505

Closed
janbartel opened this issue Jan 21, 2020 · 0 comments · Fixed by #4509
Closed

Servlet 4 get/setInitParameter() and get/setAttribute() must throw NPE if arg is null #4505

janbartel opened this issue Jan 21, 2020 · 0 comments · Fixed by #4509
Assignees
@janbartel
Labels
Specification For all industry Specifications (IETF / Servlet / etc)

Comments

@janbartel
Copy link
Contributor

jetty-10

Servlet Spec 4, Changes Since 3.1, page A-203:

  1. Modify javadoc for ServletContext getAttribute() and getInitParameter() ,
    specify that NullPointerException must be thrown if the argument “name” is
    null.
  2. Modify javadoc for ServletContext.setAttribute() and setInitParameter()
    to specify that NullPointerException must be thrown if the “name” argument is
    null.
@janbartel janbartel added the Specification For all industry Specifications (IETF / Servlet / etc) label Jan 21, 2020
@janbartel janbartel self-assigned this Jan 21, 2020
janbartel added a commit that referenced this issue Jan 21, 2020
@janbartel
Issue #4505 Ensure get/setInitParam with null arg throws NPE
7c6d717 
Signed-off-by: Jan Bartel <janb@webtide.com>
@janbartel janbartel mentioned this issue Jan 21, 2020
Merged
janbartel added a commit that referenced this issue Jan 22, 2020
@janbartel
Issue #4505 Changes after review
29ea097 
Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jan 22, 2020
@janbartel
Issue #4505 Ensure get/setInitParam with null arg throws NPE (#4509)
158789b 
* Issue #4505 Ensure get/setInitParam with null arg throws NPE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@janbartel janbartel

Labels
Specification For all industry Specifications (IETF / Servlet / etc)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #4505 Ensure get/setInitParam with null arg throws NPE jetty/jetty.project
1 participant
@janbartel