add support for JSON.jl 1.x

This adds support for JSON.jl 1.x. The package will now be compatible with both 0.x and 1.x versions of JSON.

Author: tanmaykm

Project.toml

@@ -4,7 +4,7 @@ keywords = ["Swagger", "OpenAPI", "REST"]
 license = "MIT"
 desc = "OpenAPI server and client helper for Julia"
 authors = ["JuliaHub Inc."]
-version = "0.2.1"
+version = "0.2.2"
 
 [deps]
 Base64 = "2a0f44e3-6c83-55bd-87e4-b1978d98bd5f"
@@ -22,7 +22,7 @@ p7zip_jll = "3f19e933-33d8-53b3-aaab-bd5110c3b7a0"
 [compat]
 Downloads = "1"
 HTTP = "1"
-JSON = "0.20, 0.21"
+JSON = "0.20, 0.21, 1"
 LibCURL = "0.6"
 MIMEs = "0.1, 1"
 MbedTLS = "0.6.8, 0.7, 1"

src/OpenAPI.jl

@@ -7,6 +7,15 @@ using p7zip_jll
 import Base: getindex, keys, length, iterate, hasproperty
 import JSON: lower
 
+
+const _JSON_PARSE_ALLOW_TRAILING = try; JSON.parse("1 "; allowtrailing=true); true; catch; false; end
+
+if _JSON_PARSE_ALLOW_TRAILING
+    _json_parse(io_or_str) = JSON.parse(io_or_str; allowtrailing=true)
+else
+    _json_parse(io_or_str) = JSON.parse(io_or_str)
+end
+
 include("commontypes.jl")
 include("datetime.jl")
 include("val.jl")

src/client.jl

@@ -12,7 +12,7 @@ using MIMEs
 
 import Base: convert, show, summary, getproperty, setproperty!, iterate
 import ..OpenAPI: APIModel, UnionAPIModel, OneOfAPIModel, AnyOfAPIModel, APIClientImpl, OpenAPIException, InvocationException, to_json, from_json, validate_property, property_type
-import ..OpenAPI: str2zoneddatetime, str2datetime, str2date
+import ..OpenAPI: str2zoneddatetime, str2datetime, str2date, _json_parse
 
 include("client/clienttypes.jl")
 include("client/chunk_readers.jl")
@@ -159,8 +159,8 @@ response(::Type{DateTime}, data) = str2datetime(data)
 response(::Type{Date}, data) = str2date(data)
 
 response(::Type{T}, data) where {T} = convert(T, data)
-response(::Type{T}, data::Dict{String,Any}) where {T} = from_json(T, data)::T
-response(::Type{T}, data::Dict{String,Any}) where {T<:Dict} = convert(T, data)
+response(::Type{T}, data::AbstractDict{String,Any}) where {T} = from_json(T, data)::T
+response(::Type{T}, data::AbstractDict{String,Any}) where {T<:Dict} = convert(T, Dict{String,Any}(data))
 response(::Type{Vector{T}}, data::Vector{V}) where {T,V} = T[response(T, v) for v in data]
 
 noop_pre_request_hook(ctx::Ctx) = ctx
@@ -299,14 +299,14 @@ end
 
 Base.hasproperty(o::T, name::Symbol) where {T<:APIModel} = ((name in propertynames(o)) && (getproperty(o, name) !== nothing))
 
-convert(::Type{T}, json::Dict{String,Any}) where {T<:APIModel} = from_json(T, json)
+convert(::Type{T}, json::AbstractDict{String,Any}) where {T<:APIModel} = from_json(T, json)
 convert(::Type{T}, v::Nothing) where {T<:APIModel} = T()
 convert(::Type{T}, v::T) where {T<:OneOfAPIModel} = v
-convert(::Type{T}, json::Dict{String,Any}) where {T<:OneOfAPIModel} = from_json(T, json)
+convert(::Type{T}, json::AbstractDict{String,Any}) where {T<:OneOfAPIModel} = from_json(T, json)
 convert(::Type{T}, v) where {T<:OneOfAPIModel} = T(v)
 convert(::Type{T}, v::String) where {T<:OneOfAPIModel} = T(v)
 convert(::Type{T}, v::T) where {T<:AnyOfAPIModel} = v
-convert(::Type{T}, json::Dict{String,Any}) where {T<:AnyOfAPIModel} = from_json(T, json)
+convert(::Type{T}, json::AbstractDict{String,Any}) where {T<:AnyOfAPIModel} = from_json(T, json)
 convert(::Type{T}, v) where {T<:AnyOfAPIModel} = T(v)
 convert(::Type{T}, v::String) where {T<:AnyOfAPIModel} = T(v)
 

src/client/chunk_readers.jl

@@ -34,7 +34,7 @@ function Base.iterate(iter::JSONChunkReader, _state=nothing)
             end
         end
         eof(iter.buffered_input) && return nothing
-        valid_json = JSON.parse(iter.buffered_input)
+        valid_json = _json_parse(iter.buffered_input)
         bytes = convert(Vector{UInt8}, codeunits(JSON.json(valid_json)))
         return (bytes, iter)
     end

src/json.jl

@@ -42,7 +42,7 @@ end
 
 is_deep_explode(sctx::StyleCtx) = sctx.name == "deepObject" && sctx.is_explode
 
-function deep_object_to_array(src::Dict)
+function deep_object_to_array(src::AbstractDict)
     keys_are_int = all(key -> occursin(r"^\d+$", key), keys(src))
     if keys_are_int
         sorted_keys = sort(collect(keys(src)), by=x->parse(Int, x))
@@ -58,14 +58,14 @@ end
 
 to_json(o) = JSON.json(o)
 
-from_json(::Type{Union{Nothing,T}}, json::Dict{String,Any}; stylectx=nothing) where {T} = from_json(T, json; stylectx)
-from_json(::Type{T}, json::Dict{String,Any}; stylectx=nothing) where {T} = from_json(T(), json; stylectx)
-from_json(::Type{T}, json::Dict{String,Any}; stylectx=nothing) where {T <: Dict} = convert(T, json)
-from_json(::Type{T}, j::Dict{String,Any}; stylectx=nothing) where {T <: String} = to_json(j)
-from_json(::Type{Any}, j::Dict{String,Any}; stylectx=nothing) = j
+from_json(::Type{Union{Nothing,T}}, json::AbstractDict{String,Any}; stylectx=nothing) where {T} = from_json(T, json; stylectx)
+from_json(::Type{T}, json::AbstractDict{String,Any}; stylectx=nothing) where {T} = from_json(T(), json; stylectx)
+from_json(::Type{T}, json::AbstractDict{String,Any}; stylectx=nothing) where {T <: Dict} = convert(T, Dict{String,Any}(json))
+from_json(::Type{T}, j::AbstractDict{String,Any}; stylectx=nothing) where {T <: String} = to_json(j)
+from_json(::Type{Any}, j::AbstractDict{String,Any}; stylectx=nothing) = j
 from_json(::Type{Vector{T}}, j::Vector{Any}; stylectx=nothing) where {T} = j
 
-function from_json(::Type{Vector{T}}, json::Dict{String, Any}; stylectx=nothing) where {T}
+function from_json(::Type{Vector{T}}, json::AbstractDict{String, Any}; stylectx=nothing) where {T}
     if !isnothing(stylectx) && is_deep_explode(stylectx)
         cvt = deep_object_to_array(json)
         if isa(cvt, Vector)
@@ -78,7 +78,7 @@ function from_json(::Type{Vector{T}}, json::Dict{String, Any}; stylectx=nothing)
     end
 end
 
-function from_json(o::T, json::Dict{String,Any};stylectx=nothing) where {T <: UnionAPIModel}
+function from_json(o::T, json::AbstractDict{String,Any};stylectx=nothing) where {T <: UnionAPIModel}
     return from_json(o, :value, json;stylectx)
 end
 
@@ -88,16 +88,16 @@ function from_json(o::T, val::Union{String,Real};stylectx=nothing) where {T <: U
     return o
 end
 
-function from_json(o::T, json::Dict{String,Any};stylectx=nothing) where {T <: APIModel}
+function from_json(o::T, json::AbstractDict{String,Any};stylectx=nothing) where {T <: APIModel}
     jsonkeys = [Symbol(k) for k in keys(json)]
     for name in intersect(propertynames(o), jsonkeys)
         from_json(o, name, json[String(name)];stylectx)
     end
     return o
 end
 
-function from_json(o::T, name::Symbol, json::Dict{String,Any};stylectx=nothing) where {T <: APIModel}
-    ftype = (T <: UnionAPIModel) ? property_type(T, name, json) : property_type(T, name)
+function from_json(o::T, name::Symbol, json::AbstractDict{String,Any};stylectx=nothing) where {T <: APIModel}
+    ftype = (T <: UnionAPIModel) ? property_type(T, name, Dict{String,Any}(json)) : property_type(T, name)
     fval = from_json(ftype, json; stylectx)
     setfield!(o, name, convert(ftype, fval))
     return o

src/server.jl

@@ -89,7 +89,7 @@ to_param_type(::Type{Vector{UInt8}}, val::String; stylectx=nothing) = convert(Ve
 to_param_type(::Type{Vector{T}}, val::Vector{T}, _collection_format::Union{String,Nothing}; stylectx=nothing) where {T} = val
 to_param_type(::Type{Vector{T}}, json::Vector{Any}; stylectx=nothing) where {T} = [to_param_type(T, x; stylectx) for x in json]
 
-function to_param_type(::Type{Vector{T}}, json::Dict{String, Any}; stylectx=nothing) where {T}
+function to_param_type(::Type{Vector{T}}, json::AbstractDict{String, Any}; stylectx=nothing) where {T}
     if !isnothing(stylectx) && is_deep_explode(stylectx)
         cvt = deep_object_to_array(json)
         if isa(cvt, Vector)
@@ -103,7 +103,7 @@ function to_param_type(::Type{T}, strval::String; stylectx=nothing) where {T <:
     from_json(T, JSON.parse(strval); stylectx)
 end
 
-function  to_param_type(::Type{T}, json::Dict{String,Any}; stylectx=nothing) where {T <: APIModel}
+function  to_param_type(::Type{T}, json::AbstractDict{String,Any}; stylectx=nothing) where {T <: APIModel}
     from_json(T, json; stylectx)
 end