Skip to content

Extend XML validation feature#2225

Open
jens-st wants to merge 2 commits intoartefactual:qa/1.xfrom
jens-st:dev/issue-1786-extend-xml-validation
Open

Extend XML validation feature#2225
jens-st wants to merge 2 commits intoartefactual:qa/1.xfrom
jens-st:dev/issue-1786-extend-xml-validation

Conversation

@jens-st
Copy link

@jens-st jens-st commented Mar 5, 2026

archivematica/Issues#1786

  • Forward the metadata XML validation dictionary to the resolver when parsing XSD schemas themselves.
  • Always check the dictionary first before falling back to resolving the original schemaLocation.
  • Implement a timeout for schema URL requests.

This change should provide a way forward for broken external schema resources, potentially prevent being rate-limited or blocked by external sites, and overall stop or heavily reduce outgoing schema-related traffic (while possibly decreasing loading times). Additionally, it will enable the user to have a more controlled local schema file catalog for security concerns.

- Forward the metadata XML validation dictionary to the resolver when parsing XSD schemas themselves.
- Always check the dictionary first before falling back to resolving the original schemaLocation.
- Implement a timeout for schema URL requests.

This change should provide a way forward for broken external schema resources, potentially prevent being rate-limited or blocked by external sites, and overall stop or heavily reduce outgoing schema-related traffic (while possibly decreasing loading times). Additionally, it will enable the user to have a more controlled local schema file catalog for security concerns.
@replaceafill replaceafill added the Community Pull requests that have been contributed from community members outside Artefactual label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Community Pull requests that have been contributed from community members outside Artefactual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants