Skip to content

CPDH "releaseInfo" can be empty#670

Merged
JDanielSmith merged 9 commits intomainfrom
feature/CPHD-1.1.0
Jul 5, 2023
Merged

CPDH "releaseInfo" can be empty#670
JDanielSmith merged 9 commits intomainfrom
feature/CPHD-1.1.0

Conversation

@JDanielSmith
Copy link
Copy Markdown

... we found that the six library does a check when reading a CPHD file to ensure that the Release Info field not only exists, but that it is not an empty string. The CPHD spec does require that the field exists, but does not seem to prohibit it from being an empty string.

We have gotten some vendor CPHDs with an empty field here and have failed reading the file due to this. I checked and SARPY is able to read (it must not implement the same check). ...

Loading
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.

1 participant