Skip to content

[5.x]: Modify XML Schema and DTD URLs for Unidata hostname switch#916

Merged
haileyajohnson merged 3 commits intoUnidata:maint-5.xfrom
ethanrd:fix-changed-schema-urls
Oct 29, 2021
Merged

[5.x]: Modify XML Schema and DTD URLs for Unidata hostname switch#916
haileyajohnson merged 3 commits intoUnidata:maint-5.xfrom
ethanrd:fix-changed-schema-urls

Conversation

@ethanrd
Copy link
Copy Markdown
Member

@ethanrd ethanrd commented Oct 28, 2021

Description of Changes

Unidata hosted XML Schema and DTD files are now available from "https://schemas.unidata.ucar.edu/" (with redirects in place from the old URLs, "http://www.unidata.ucar.edu/schemas/"). This change updates all occurrences.

PR Checklist

  • Indicate the version associated with this PR in the Title
  • Link to any issues that the PR addresses
  • Add labels, especially if the PR should be ported to other versions
  • Open as a draft PR
    until ready for review
  • Make sure GitHub tests pass
  • Mark PR as "Ready for Review"

@ethanrd ethanrd added the ver5 version 5 only. label Oct 28, 2021
@ethanrd
Copy link
Copy Markdown
Member Author

ethanrd commented Oct 29, 2021

CatalogGenConfig.*.xsd is referenced in some cdm-test files and in legacy code but is itself no longer in resources/thredds/schema directory. Could consider adding it to legacy/src/main/resources/resources/thredds/schemas/ but probably not needed.

The legacy module might be a better place for queryCapability.0.4.xsd as well. But would have to make sure those resources make it into ncIDV.jar. Probably not worth effort at the moment.

I will get this PR ready for review shortly.

@ethanrd ethanrd marked this pull request as ready for review October 29, 2021 17:24
@haileyajohnson haileyajohnson merged commit 75553c1 into Unidata:maint-5.x Oct 29, 2021
@ethanrd ethanrd deleted the fix-changed-schema-urls branch November 1, 2021 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ver5 version 5 only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants