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

XML Catalog GF/Payara: Ensure only existing schemas are resolved #7218

Merged

Conversation

matthiasblaesing
Copy link
Contributor

Before this change the InputSource for some of the schema referenced by the JSF files failed to load when GF 5.X was registered. The problem was, that the Catalog code resolved jakartaee_web_services_client_2_0.xsd to a non existing local path and in turn the XML parser failed to parse the JSF component libraries.

Closes: #7137

Before this change the InputSource for some of the schema referenced by
the JSF files failed to load when GF 5.X was registered. The problem
was, that the Catalog code resolved jakartaee_web_services_client_2_0.xsd
to a non existing local path and in turn the XML parser failed to parse
the JSF component libraries.

Closes: apache#7137
@matthiasblaesing matthiasblaesing added enterprise [ci] enable enterprise job ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Apr 1, 2024
@matthiasblaesing matthiasblaesing added this to the NB22 milestone Apr 1, 2024
@ebarboni
Copy link
Contributor

ebarboni commented Apr 9, 2024

LGTM syntax validation looks good

@matthiasblaesing
Copy link
Contributor Author

@ebarboni thanks for the review!

@matthiasblaesing matthiasblaesing merged commit dcd15da into apache:master Apr 9, 2024
34 checks passed
@matthiasblaesing matthiasblaesing deleted the catalogs_payara_glassfish branch April 19, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) enterprise [ci] enable enterprise job
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Faces syntax validation fails for EE-10
2 participants