Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

json validation cleanup, part 1 #1066

Merged
merged 7 commits into from
Nov 19, 2020
Merged

json validation cleanup, part 1 #1066

merged 7 commits into from
Nov 19, 2020

Commits on Nov 19, 2020

  1. render these validation failure log messages nicer

    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    11384e1 View commit details
    Browse the repository at this point in the history

  2. add a few more basic checks for http standards conformance

    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    dac1847 View commit details
    Browse the repository at this point in the history

  3. remove numeric coercion in query_params validation 

    We can just define our data in terms of the original strings.
    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    93ff391 View commit details
    Browse the repository at this point in the history

  4. apply defaults when parsing query parameters 

    Note: it is *not* correct to use default coercions within JSON::Validator, as
its usage violates the spec for how to apply annotations (for example, a
missing property cannot have an annotation and therefore cannot have a
default: default values must be specified at the object level for each
property, rather than in the individual properties' subschemas).
    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    bf67241 View commit details
    Browse the repository at this point in the history

  5. eliminate some unneeded local variables

    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    ef5556f View commit details
    Browse the repository at this point in the history

  6. support $t->json_schema_is('http...')

    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    b03734b View commit details
    Browse the repository at this point in the history

  7. s/definitions/$defs/ in all JSON schemas

    @karenetheridge
    karenetheridge committed Nov 19, 2020
    Configuration menu
    Copy the full SHA
    f308f34 View commit details
    Browse the repository at this point in the history