Skip to content

ICD REST JSON and several message examples have syntax or field-order errors #18

Description

@mfellner

ICD v1.4.6 REST GET (p30) repeats four message properties in one object, losing messages under last-key-wins JSON parsing. POST (p31) repeats a property and omits a comma. REST payloads also use outdated field layouts. METEO Sample 1 (p13) uses 33,RefPoint1 instead of two semicolon-separated fields; METEOTest silently uses a corrected sample. COMMAND samples 1/2 (p19) place values in the wrong header/content positions. The compression reference on p3 also has an extra empty field before TEXT type 1, so both the reference and its compressed bytes need updating.

Reference: source at reviewed commit; ICD v1.4.6.

Proposed correction: Correct the published JSON and message layouts, provide literal validated fixtures with decoded-field assertions, and keep DOCX/PDF synchronized. Coordinate sequence-number and GRAPHIC notation edits with their dedicated issues to avoid conflicting example changes.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions