-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
DeprecationWarning Decorator #3764
Comments
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. |
Stop / Start is not typical behavior. The I say we don't bother with stop/start behavior and leave it like it is. |
Add in doStart Signed-off-by: Greg Wilkins <[email protected]>
…rning Fix #3764 DeprecationWarning decorator
The
DeprecationWarning
decorator is always added to theDecoratedObjectFactory
in the constructor ofServletContextHandler
.However, over a stop/start cycle the factory is cleared and the decorator is not re-added.
Moreover, is it really our responsibility to always warn if a deprecated servlet is used? Could we not make this optional? Perhaps in jetty-10 we can have a DeprecatedConfiguration that will add this decorator in pre-configure (and thus will work over stop/start)?
The text was updated successfully, but these errors were encountered: