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

Review why ServletRequestState#asyncError is never called in Jetty-12 #9060

Closed
lachlan-roberts opened this issue Dec 16, 2022 · 2 comments
Closed

Review why ServletRequestState#asyncError is never called in Jetty-12 #9060

lachlan-roberts opened this issue Dec 16, 2022 · 2 comments
Assignees
@lachlan-roberts
Labels
Stale For auto-closed stale issues and pull requests

Comments

@lachlan-roberts
Copy link
Contributor

Enhancement Description

@lachlan-roberts can you also review org.eclipse.jetty.ee10.servlet.ServletRequestState#asyncError which is not called. Need to check if that is correct and that onError is correctly called when necessary.

Originally posted by @gregw in #9048 (comment)
@lachlan-roberts lachlan-roberts self-assigned this Dec 16, 2022
@lachlan-roberts lachlan-roberts mentioned this issue Dec 16, 2022
Merged
@github-actions GitHub Actions
Copy link

This issue has been automatically marked as stale because it has been a
full year without activity. It will be closed if no further activity occurs.
Thank you for your contributions.

@github-actions github-actions bot added the Stale For auto-closed stale issues and pull requests label Dec 17, 2023
@sbordet
Copy link
Contributor

sbordet commented Dec 17, 2023

Closing as this issue has been handled in #10933 and #10949.

@sbordet sbordet closed this as completed Dec 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lachlan-roberts lachlan-roberts

Labels
Stale For auto-closed stale issues and pull requests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sbordet @lachlan-roberts