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.

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

java.lang.NullPointerException: null error on WebDAV Domain due to jetty 9.4.51 bug #7631

Closed
geonmo opened this issue Aug 3, 2024 · 3 comments

Comments

@geonmo
Copy link

geonmo commented Aug 3, 2024

Dear dCache staff,

I found some errors on a WebDAV domain.

2024-08-04T00:36:58.908509+09:00 cms-t2-se01.sdfarm.kr dcache@WebDavDomain[488477]: java.lang.NullPointerException: null
2024-08-04T00:36:58.908509+09:00 cms-t2-se01.sdfarm.kr dcache@WebDavDomain[488477]: #011at org.eclipse.jetty.io.AbstractConnection.lambda$failedCallback$0(AbstractConnection.java:92)
2024-08-04T00:36:58.908509+09:00 cms-t2-se01.sdfarm.kr dcache@WebDavDomain[488477]: #011at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:883)
2024-08-04T00:36:58.908509+09:00 cms-t2-se01.sdfarm.kr dcache@WebDavDomain[488477]: #011at org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:1034)
2024-08-04T00:36:58.908509+09:00 cms-t2-se01.sdfarm.kr dcache@WebDavDomain[488477]: #011at java.base/java.lang.Thread.run(Thread.java:829)

I don't know if it's due to this bug, but if the number of failed transfers at the WebDAV door increases, at some point the WebDAV domain will stop working and even basic functions like serving SSL certificates won't work.

I googled the error and found that it occurred in jetty 9.4.51 and was fixed in 9.4.52. (jetty/jetty.project#9476)

However, I understand that dCache v9.2 uses jetty 9.4.51.

I strongly request that the jetty version upgrade in the next release of dCache v9.2.

Please also check whether this bug can cause any abnormal behavior of a WebDAV domain.

The webdav hang error has significantly reduced the availability of our site.

Regards,

-- Geonmo

@kofemann
Copy link
Member

kofemann commented Aug 5, 2024

Hi @geonmo ,

Thanks for reporting. Do you know how to reproduce the issue?

@geonmo
Copy link
Author

geonmo commented Aug 6, 2024

Hello, @kofemann ,

The code to test jetty itself is in issue #9476. (jetty/jetty.project#9476)

In my case, I recently experienced a data upload issue slowdown that brought the issue.

Currently on our site, roughly 450 uploads are consistently failing with timeouts, and the webdav domain is down every 8 hours, as short as 1 hour and as long as 8 hours. I'm assuming that not that many are failing because the timeout time (7200) is long.

I think it might be possible to make the timeout of the webdav transfer task smaller and then test it to keep failing.

kofemann pushed a commit to kofemann/dcache that referenced this issue Aug 26, 2024
Bumps [org.eclipse.jetty:jetty-servlets](https://github.com/eclipse/jetty.project) from 9.4.51.v20230217 to 9.4.52.v20230823.
- [Release notes](https://github.com/eclipse/jetty.project/releases)
- [Commits](jetty/jetty.project@jetty-9.4.51.v20230217...jetty-9.4.52.v20230823)

---
updated-dependencies:
- dependency-name: org.eclipse.jetty:jetty-servlets
  dependency-type: direct:production
...

Fixes: dCache#7631

Signed-off-by: dependabot[bot] <[email protected]>
(cherry picked from commit 43a3e44)
Signed-off-by: Tigran Mkrtchyan <[email protected]>
mksahakyan pushed a commit that referenced this issue Aug 28, 2024
Bumps [org.eclipse.jetty:jetty-servlets](https://github.com/eclipse/jetty.project) from 9.4.51.v20230217 to 9.4.52.v20230823.
- [Release notes](https://github.com/eclipse/jetty.project/releases)
- [Commits](jetty/jetty.project@jetty-9.4.51.v20230217...jetty-9.4.52.v20230823)

---
updated-dependencies:
- dependency-name: org.eclipse.jetty:jetty-servlets
  dependency-type: direct:production
...

Fixes: #7631

Signed-off-by: dependabot[bot] <[email protected]>
(cherry picked from commit 43a3e44)
Signed-off-by: Tigran Mkrtchyan <[email protected]>
@geonmo
Copy link
Author

geonmo commented Sep 6, 2024

I've seen the release with the jetty version change, thank you for taking care of it. I'll close this issue.

@geonmo geonmo closed this as completed Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants