Skip to content

Conversation

@volosied
Copy link
Contributor

@volosied volosied commented Feb 6, 2025

As described in the comment:

 * ... the listener checks if 
 * the FacesServlet has already been defined in web.xml and if not, it adds
 * the FacesServlet with the mappings (/faces/*, *.jsf, *.faces) dynamically.

In summary, if this annotation is found and the web.xml doesn't have any mappings listed for the faces servlet, they will be added automatically.

I'm assuming this was missed when this annotation was added 2.3

@tandraschko tandraschko merged commit 90eb2ec into apache:main Feb 7, 2025
4 checks passed
@melloware
Copy link
Contributor

Does this need to be backported?

@volosied
Copy link
Contributor Author

volosied commented Feb 7, 2025

Yes, I'll do that soon.

@melloware
Copy link
Contributor

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants