Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Allow NcML with https in URI #794

Merged
merged 1 commit into from
Apr 4, 2017
Merged

Allow NcML with https in URI #794

merged 1 commit into from
Apr 4, 2017

Conversation

lesserwhirls
Copy link
Collaborator

Enable the use of https in the URI for the NcML namespace when reading NcML files.

For example, this works as it should:

<netcdf xmlns="http://www.unidata.ucar.edu/namespaces/netcdf/ncml-2.2">

but this fails:

<netcdf xmlns="https://www.unidata.ucar.edu/namespaces/netcdf/ncml-2.2">

with the following:

Exception in thread "main" java.lang.IllegalArgumentException: Incorrect namespace specified in NcML= https://www.unidata.ucar.edu/namespaces/netcdf/ncml-2.2
   must be=http://www.unidata.ucar.edu/namespaces/netcdf/ncml-2.2
        at ucar.nc2.ncml.NcMLReader.readNetcdf(NcMLReader.java:489)
        at ucar.nc2.ncml.NcMLReader._readNcML(NcMLReader.java:456)
        at ucar.nc2.ncml.NcMLReader.readNcML(NcMLReader.java:256)
        at ucar.nc2.ncml.NcMLReader.readNcML(NcMLReader.java:206)
        at ucar.nc2.dataset.NetcdfDataset.acquireNcml(NetcdfDataset.java:1081)
        at ucar.nc2.dataset.NetcdfDataset.openOrAcquireFile(NetcdfDataset.java:723)
        at ucar.nc2.dataset.NetcdfDataset.openFile(NetcdfDataset.java:572)
        at ucar.nc2.NCdumpW.print(NCdumpW.java:251)

Enable the use of https in the URI for the NcML namespace when reading NcML files
@cwardgar cwardgar merged commit 11d3260 into Unidata:master Apr 4, 2017
@lesserwhirls lesserwhirls deleted the ncml branch April 7, 2017 01:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants