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.

Sign up for GitHub

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

Jump to bottom

Fix handling of '/' characters in names in DAP2. #1319

Merged
merged 1 commit into from
Feb 15, 2019
Merged

Fix handling of '/' characters in names in DAP2. #1319

merged 1 commit into from
Feb 15, 2019

Commits on Feb 15, 2019

  1. Fix handling of '/' characters in names in DAP2. 

    re: Unidata/thredds#1224
[note that this is an issue in thredds, but the fix is in netcdf-c]

A thredds server can encode a netcdf-4 file into DAP2
by flattening names to include the containing group path,
where the group names are separated by '/'.

But the '/' is prohibited in netcdf names even if escaped
(a decision before my time).

So, if the netcdf-c/libdap2 code encounters a DAP2 name with '/'
characters, the '/' characters are converted to the string
%2f. Unfortunately, there is a glitch, namely that converting
the leading '/' produces a name that is still illegal. This PR
modifies the code to just drop the leading '/' character.
    @DennisHeimbigner
    DennisHeimbigner committed Feb 15, 2019
    Configuration menu
    Copy the full SHA
    c59d5ce View commit details
    Browse the repository at this point in the history