Skip to content

Conversation

Artur-
Copy link
Contributor

@Artur- Artur- commented Dec 3, 2021

Fixes #28854

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Dec 3, 2021
@mbhave mbhave added type: regression A regression from a previous release and removed status: waiting-for-triage An issue we've not yet triaged labels Dec 3, 2021
@mbhave mbhave added this to the 2.6.2 milestone Dec 3, 2021
@wilkinsona wilkinsona changed the title Make ErrorPageSecurityFilter compatible with Servlet 3 AbstractMethodError in org.springframework.boot.web.servlet.filter.ErrorPageSecurityFilter when deployed to a Servlet 3.1-compatible container Dec 3, 2021
@wilkinsona wilkinsona self-assigned this Dec 9, 2021
@wilkinsona
Copy link
Member

@Artur- Thanks very much for making your first contribution to Spring Boot.

@mfusek
Copy link

mfusek commented Mar 8, 2022

void destroy() should be also implemented. Now tomcat logs
13:25:18.075 [/xyz] {localhost-startStop-12} ERROR : Failed to destroy the filter named [errorPageSecurityFilter] of type [org.springframework.boot.web.servlet.filter.ErrorPageSecurityFilter]
java.lang.AbstractMethodError: null
at org.apache.catalina.core.ApplicationFilterConfig.release(ApplicationFilterConfig.java:312)
at org.apache.catalina.core.StandardContext.filterStop(StandardContext.java:4642)
at org.apache.catalina.core.StandardContext.stopInternal(StandardContext.java:5459)
at org.apache.catalina.util.LifecycleBase.stop(LifecycleBase.java:257)
at org.apache.catalina.core.ContainerBase$StopChild.call(ContainerBase.java:1443)
at org.apache.catalina.core.ContainerBase$StopChild.call(ContainerBase.java:1432)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:750)

@wilkinsona
Copy link
Member

@mfusek #29558 has addressed that. Please upgrade to 2.6.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: regression A regression from a previous release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants