Skip to content

Fix geowebcache.xml validation problems - #1610

Open
jodygarnett wants to merge 4 commits into
GeoWebCache:mainfrom
jodygarnett:config-xsl-namespace
Open

jodygarnett wants to merge 4 commits into
GeoWebCache:mainfrom
jodygarnett:config-xsl-namespace

Conversation

@jodygarnett

@jodygarnett jodygarnett commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

I am so tired of validation errors on startup:

  • Fix geowebcache_200.xsl dropping the namespace when upgrading 1.x config
  • Test backwards compatibility with a whole bunch of prior files just to be sure.
  • Check for similar glitches in prior xsl / xsd
  • Introduce GeoWebCacheXMLUpdate to update and sort elements with comments
  • Try re-update on validation fail to catch simple wrong order mistakes

aside: this removes the start warnings seen in GS3 (and earlier...)

…figurations

Test backwards compatibility with a whole bunch of prior files
Gltiches:
- Correct geowebcache_124.xsd namespace
- Note 1.5.1/1.6.1 schema namespace released incorrectly
@jodygarnett jodygarnett changed the title Fix geowebcache_200.xsl dropping the namespace when upgrading 1.x config Fix geowebcache.xml validation problems Oct 29, 2026
This is used during load, and can also be used to try and recover after simple validation fail due to order differences.
@jodygarnett
jodygarnett requested a balanced review from Copilot October 29, 2026 20:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@jodygarnett
jodygarnett requested a review from groldan October 29, 2026 20:56
@petersmythe
petersmythe requested a max review from Copilot October 30, 2026 10:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@jodygarnett jodygarnett left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to keep this self contained, however it is generally useful it could be busted out as a GeoTools XMLUtils utility class.

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.

2 participants