Merge pull request #109 from JuliaWeb/KristofferC-patch-2

use idle_timeout for the requests

.travis.yml

@@ -1,4 +1,7 @@
 language: julia
+os:
+  - linux
+  - osx
 julia:
     - 0.6
     - nightly

REQUIRE

@@ -1,6 +1,6 @@
 julia 0.6
 
-Compat 0.43
+Compat 0.62
 JSON
 MbedTLS
 HTTP 0.6.3

src/activity/events.jl

@@ -43,7 +43,7 @@ sig_header(request::HTTP.Request) = HTTP.header(request, "X-Hub-Signature")
 
 function has_valid_secret(request::HTTP.Request, secret)
     if has_sig_header(request)
-        secret_sha = "sha1="*bytes2hex(MbedTLS.digest(MbedTLS.MD_SHA1, HTTP.payload(request, String), secret))
+        secret_sha = "sha1="*bytes2hex(MbedTLS.digest(MbedTLS.MD_SHA1, HTTP.payload(request), secret))
         return sig_header(request) == secret_sha
     end
     return false
@@ -97,7 +97,7 @@ function handle_event_request(request, handle;
         return HTTP.Response(204, "event ignored")
     end
 
-    event = event_from_payload!(event_header(request), JSON.parse(HTTP.payload(request, String)))
+    event = event_from_payload!(event_header(request), JSON.parse(IOBuffer(HTTP.payload(request))))
 
     if !(isa(repos, Nothing)) && !(from_valid_repo(event, repos))
         return HTTP.Response(400, "invalid repo")

src/git/reference.jl

@@ -10,12 +10,12 @@ name(ref::Reference) = String(split(get(ref.ref), "refs/")[2])
 
 @api_default function reference(api::GitHubAPI, repo, ref_obj; options...)
     result = gh_get_json(api, "/repos/$(name(repo))/git/refs/$(name(ref_obj))"; options...)
-    return Reference.(result)
+    return Reference(result)
 end
 
 @api_default function references(api::GitHubAPI, repo; options...)
     results, page_data = gh_get_paged_json(api, "/repos/$(name(repo))/git/refs"; options...)
-    return Reference.(results), page_data
+    return Reference.((results,)), page_data
 end
 
 @api_default function create_reference(api::GitHubAPI, repo; options...)

src/utils/GitHubType.jl

@@ -82,7 +82,7 @@ end
 @generated function json2github(::Type{G}, data::Dict) where {G<:GitHubType}
     types = G.types
     fields = fieldnames(G)
-    args = Vector{Expr}(length(fields))
+    args = Vector{Expr}(undef, length(fields))
     for i in eachindex(fields)
         field, T = fields[i], first(types[i].parameters)
         key = field == :typ ? "type" : string(field)
@@ -125,27 +125,27 @@ end
 ###################
 
 function Base.show(io::IO, g::GitHubType)
-    print(io, "$(typeof(g)) (all fields are Nullable):")
-    for field in fieldnames(typeof(g))
-        val = getfield(g, field)
-        if !(isnull(val))
-            gotval = get(val)
-            println(io)
-            print(io, "  $field: ")
-            if isa(gotval, Vector)
-                print(io, typeof(gotval))
-            else
-                showcompact(io, gotval)
-            end
+    if get(io, :compact, false)
+        uri_id = namefield(g)
+        if isnull(uri_id)
+            print(io, typeof(g), "(…)")
+        else
+            print(io, typeof(g), "($(repr(get(uri_id))))")
         end
-    end
-end
-
-function Base.showcompact(io::IO, g::GitHubType)
-    uri_id = namefield(g)
-    if isnull(uri_id)
-        print(io, typeof(g), "(…)")
     else
-        print(io, typeof(g), "($(repr(get(uri_id))))")
+        print(io, "$(typeof(g)) (all fields are Nullable):")
+        for field in fieldnames(typeof(g))
+            val = getfield(g, field)
+            if !(isnull(val))
+                gotval = get(val)
+                println(io)
+                print(io, "  $field: ")
+                if isa(gotval, Vector)
+                    print(io, typeof(gotval))
+                else
+                    show(IOContext(io, :compact => true), gotval)
+                end
+            end
+        end
     end
 end

src/utils/requests.jl

@@ -56,9 +56,9 @@ function github_request(api::GitHubAPI, request_method, endpoint;
     _headers = convert(Dict{String, String}, headers)
     !haskey(_headers, "User-Agent") && (_headers["User-Agent"] = "GitHub-jl")
     if request_method == HTTP.get
-        r = request_method(merge(api_endpoint, query = params), _headers, redirect = allowredirects, status_exception = false)
+        r = request_method(merge(api_endpoint, query = params), _headers, redirect = allowredirects, status_exception = false, idle_timeout=20)
     else
-        r = request_method(string(api_endpoint), _headers, JSON.json(params), redirect = allowredirects, status_exception = false)
+        r = request_method(string(api_endpoint), _headers, JSON.json(params), redirect = allowredirects, status_exception = false, idle_timeout=20)
     end
     handle_error && handle_response_error(r)
     return r
@@ -92,7 +92,7 @@ get_page_links(r) = split(HTTP.header(r, "Link",), ",")
 function find_page_link(links, rel)
     relstr = "rel=\"$(rel)\""
     for i in 1:length(links)
-        if contains(links[i], relstr)
+        if occursin(relstr, links[i])
             return i
         end
     end

test/commit_comment.jl

@@ -1,3 +1,5 @@
+using Compat.Base64
+
 function create_event()
     data_base64 = "eyJhY3Rpb24iOiJjcmVhdGVkIiwiY29tbWVudCI6eyJ1cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvY29tbWVudHMvMTQwNDcwNTgiLCJodG1sX3VybCI6Imh0dHBzOi8vZ2l0aHViLmNvbS9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbW1pdC8zMmQzNWYyODU3NzdiMDc3ZDhiNmEyNTIxMzA5ZDFhYjY0NmQyMzc5I2NvbW1pdGNvbW1lbnQtMTQwNDcwNTgiLCJpZCI6MTQwNDcwNTgsInVzZXIiOnsibG9naW4iOiJqcmV2ZWxzIiwiaWQiOjMyNzc0NDMsImF2YXRhcl91cmwiOiJodHRwczovL2F2YXRhcnMuZ2l0aHVidXNlcmNvbnRlbnQuY29tL3UvMzI3NzQ0Mz92PTMiLCJncmF2YXRhcl9pZCI6IiIsInVybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscyIsImh0bWxfdXJsIjoiaHR0cHM6Ly9naXRodWIuY29tL2pyZXZlbHMiLCJmb2xsb3dlcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL2ZvbGxvd2VycyIsImZvbGxvd2luZ191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvZm9sbG93aW5ney9vdGhlcl91c2VyfSIsImdpc3RzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9naXN0c3svZ2lzdF9pZH0iLCJzdGFycmVkX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9zdGFycmVkey9vd25lcn17L3JlcG99Iiwic3Vic2NyaXB0aW9uc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvc3Vic2NyaXB0aW9ucyIsIm9yZ2FuaXphdGlvbnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL29yZ3MiLCJyZXBvc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvcmVwb3MiLCJldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL2V2ZW50c3svcHJpdmFjeX0iLCJyZWNlaXZlZF9ldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL3JlY2VpdmVkX2V2ZW50cyIsInR5cGUiOiJVc2VyIiwic2l0ZV9hZG1pbiI6ZmFsc2V9LCJwb3NpdGlvbiI6bnVsbCwibGluZSI6bnVsbCwicGF0aCI6IiIsImNvbW1pdF9pZCI6IjMyZDM1ZjI4NTc3N2IwNzdkOGI2YTI1MjEzMDlkMWFiNjQ2ZDIzNzkiLCJjcmVhdGVkX2F0IjoiMjAxNS0xMC0yOFQxNzo1Njo0OVoiLCJ1cGRhdGVkX2F0IjoiMjAxNS0xMC0yOFQxNzo1Njo0OVoiLCJib2R5IjoiYFJ1bkJlbmNobWFya3MoXCJiaW5hcnlcIiwgXCJ1bmFyeVwiKWAifSwicmVwb3NpdG9yeSI6eyJpZCI6NDQxODkxMDcsIm5hbWUiOiJCZW5jaG1hcmtUcmFja2Vycy5qbCIsImZ1bGxfbmFtZSI6Ikp1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwiLCJvd25lciI6eyJsb2dpbiI6Ikp1bGlhQ0kiLCJpZCI6OTk1NzYwNCwiYXZhdGFyX3VybCI6Imh0dHBzOi8vYXZhdGFycy5naXRodWJ1c2VyY29udGVudC5jb20vdS85OTU3NjA0P3Y9MyIsImdyYXZhdGFyX2lkIjoiIiwidXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9KdWxpYUNJIiwiaHRtbF91cmwiOiJodHRwczovL2dpdGh1Yi5jb20vSnVsaWFDSSIsImZvbGxvd2Vyc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvZm9sbG93ZXJzIiwiZm9sbG93aW5nX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvSnVsaWFDSS9mb2xsb3dpbmd7L290aGVyX3VzZXJ9IiwiZ2lzdHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9KdWxpYUNJL2dpc3Rzey9naXN0X2lkfSIsInN0YXJyZWRfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9KdWxpYUNJL3N0YXJyZWR7L293bmVyfXsvcmVwb30iLCJzdWJzY3JpcHRpb25zX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvSnVsaWFDSS9zdWJzY3JpcHRpb25zIiwib3JnYW5pemF0aW9uc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvb3JncyIsInJlcG9zX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvSnVsaWFDSS9yZXBvcyIsImV2ZW50c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvZXZlbnRzey9wcml2YWN5fSIsInJlY2VpdmVkX2V2ZW50c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL0p1bGlhQ0kvcmVjZWl2ZWRfZXZlbnRzIiwidHlwZSI6Ik9yZ2FuaXphdGlvbiIsInNpdGVfYWRtaW4iOmZhbHNlfSwicHJpdmF0ZSI6ZmFsc2UsImh0bWxfdXJsIjoiaHR0cHM6Ly9naXRodWIuY29tL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwiLCJkZXNjcmlwdGlvbiI6IlRyYWNrIGEgSnVsaWEgcGFja2FnZSdzIGJlbmNobWFyayByZXN1bHRzIGFzIHBhcnQgb2YgQ0kiLCJmb3JrIjpmYWxzZSwidXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsIiwiZm9ya3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2ZvcmtzIiwia2V5c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwva2V5c3sva2V5X2lkfSIsImNvbGxhYm9yYXRvcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbGxhYm9yYXRvcnN7L2NvbGxhYm9yYXRvcn0iLCJ0ZWFtc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvdGVhbXMiLCJob29rc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvaG9va3MiLCJpc3N1ZV9ldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2lzc3Vlcy9ldmVudHN7L251bWJlcn0iLCJldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2V2ZW50cyIsImFzc2lnbmVlc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvYXNzaWduZWVzey91c2VyfSIsImJyYW5jaGVzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9icmFuY2hlc3svYnJhbmNofSIsInRhZ3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL3RhZ3MiLCJibG9ic191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvZ2l0L2Jsb2Jzey9zaGF9IiwiZ2l0X3RhZ3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC90YWdzey9zaGF9IiwiZ2l0X3JlZnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC9yZWZzey9zaGF9IiwidHJlZXNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC90cmVlc3svc2hhfSIsInN0YXR1c2VzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9zdGF0dXNlcy97c2hhfSIsImxhbmd1YWdlc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvbGFuZ3VhZ2VzIiwic3RhcmdhemVyc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvc3RhcmdhemVycyIsImNvbnRyaWJ1dG9yc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvY29udHJpYnV0b3JzIiwic3Vic2NyaWJlcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL3N1YnNjcmliZXJzIiwic3Vic2NyaXB0aW9uX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9zdWJzY3JpcHRpb24iLCJjb21taXRzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9jb21taXRzey9zaGF9IiwiZ2l0X2NvbW1pdHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2dpdC9jb21taXRzey9zaGF9IiwiY29tbWVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbW1lbnRzey9udW1iZXJ9IiwiaXNzdWVfY29tbWVudF91cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvaXNzdWVzL2NvbW1lbnRzey9udW1iZXJ9IiwiY29udGVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL2NvbnRlbnRzL3srcGF0aH0iLCJjb21wYXJlX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9jb21wYXJlL3tiYXNlfS4uLntoZWFkfSIsIm1lcmdlc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvbWVyZ2VzIiwiYXJjaGl2ZV91cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwve2FyY2hpdmVfZm9ybWF0fXsvcmVmfSIsImRvd25sb2Fkc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvZG93bmxvYWRzIiwiaXNzdWVzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9pc3N1ZXN7L251bWJlcn0iLCJwdWxsc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvcHVsbHN7L251bWJlcn0iLCJtaWxlc3RvbmVzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9taWxlc3RvbmVzey9udW1iZXJ9Iiwibm90aWZpY2F0aW9uc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3JlcG9zL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwvbm90aWZpY2F0aW9uc3s/c2luY2UsYWxsLHBhcnRpY2lwYXRpbmd9IiwibGFiZWxzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vcmVwb3MvSnVsaWFDSS9CZW5jaG1hcmtUcmFja2Vycy5qbC9sYWJlbHN7L25hbWV9IiwicmVsZWFzZXNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9yZXBvcy9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsL3JlbGVhc2Vzey9pZH0iLCJjcmVhdGVkX2F0IjoiMjAxNS0xMC0xM1QxNjoxNzowNFoiLCJ1cGRhdGVkX2F0IjoiMjAxNS0xMC0yMlQxNDo0NTowNFoiLCJwdXNoZWRfYXQiOiIyMDE1LTEwLTI4VDE3OjUyOjM2WiIsImdpdF91cmwiOiJnaXQ6Ly9naXRodWIuY29tL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwuZ2l0Iiwic3NoX3VybCI6ImdpdEBnaXRodWIuY29tOkp1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwuZ2l0IiwiY2xvbmVfdXJsIjoiaHR0cHM6Ly9naXRodWIuY29tL0p1bGlhQ0kvQmVuY2htYXJrVHJhY2tlcnMuamwuZ2l0Iiwic3ZuX3VybCI6Imh0dHBzOi8vZ2l0aHViLmNvbS9KdWxpYUNJL0JlbmNobWFya1RyYWNrZXJzLmpsIiwiaG9tZXBhZ2UiOm51bGwsInNpemUiOjM5Nywic3RhcmdhemVyc19jb3VudCI6MSwid2F0Y2hlcnNfY291bnQiOjEsImxhbmd1YWdlIjoiSnVsaWEiLCJoYXNfaXNzdWVzIjp0cnVlLCJoYXNfZG93bmxvYWRzIjp0cnVlLCJoYXNfd2lraSI6dHJ1ZSwiaGFzX3BhZ2VzIjpmYWxzZSwiZm9ya3NfY291bnQiOjEsIm1pcnJvcl91cmwiOm51bGwsIm9wZW5faXNzdWVzX2NvdW50IjoxLCJmb3JrcyI6MSwib3Blbl9pc3N1ZXMiOjEsIndhdGNoZXJzIjoxLCJkZWZhdWx0X2JyYW5jaCI6Im1hc3RlciJ9LCJvcmdhbml6YXRpb24iOnsibG9naW4iOiJKdWxpYUNJIiwiaWQiOjk5NTc2MDQsInVybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vb3Jncy9KdWxpYUNJIiwicmVwb3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9vcmdzL0p1bGlhQ0kvcmVwb3MiLCJldmVudHNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9vcmdzL0p1bGlhQ0kvZXZlbnRzIiwibWVtYmVyc191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL29yZ3MvSnVsaWFDSS9tZW1iZXJzey9tZW1iZXJ9IiwicHVibGljX21lbWJlcnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS9vcmdzL0p1bGlhQ0kvcHVibGljX21lbWJlcnN7L21lbWJlcn0iLCJhdmF0YXJfdXJsIjoiaHR0cHM6Ly9hdmF0YXJzLmdpdGh1YnVzZXJjb250ZW50LmNvbS91Lzk5NTc2MDQ/dj0zIiwiZGVzY3JpcHRpb24iOiJDb250aW5vdXMgSW50ZWdyYXRpb24gKENJKSBTdXBwb3J0IGZvciB0aGUgSnVsaWEgTGFuZ3VhZ2UifSwic2VuZGVyIjp7ImxvZ2luIjoianJldmVscyIsImlkIjozMjc3NDQzLCJhdmF0YXJfdXJsIjoiaHR0cHM6Ly9hdmF0YXJzLmdpdGh1YnVzZXJjb250ZW50LmNvbS91LzMyNzc0NDM/dj0zIiwiZ3JhdmF0YXJfaWQiOiIiLCJ1cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMiLCJodG1sX3VybCI6Imh0dHBzOi8vZ2l0aHViLmNvbS9qcmV2ZWxzIiwiZm9sbG93ZXJzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9mb2xsb3dlcnMiLCJmb2xsb3dpbmdfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL2ZvbGxvd2luZ3svb3RoZXJfdXNlcn0iLCJnaXN0c191cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvZ2lzdHN7L2dpc3RfaWR9Iiwic3RhcnJlZF91cmwiOiJodHRwczovL2FwaS5naXRodWIuY29tL3VzZXJzL2pyZXZlbHMvc3RhcnJlZHsvb3duZXJ9ey9yZXBvfSIsInN1YnNjcmlwdGlvbnNfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL3N1YnNjcmlwdGlvbnMiLCJvcmdhbml6YXRpb25zX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9vcmdzIiwicmVwb3NfdXJsIjoiaHR0cHM6Ly9hcGkuZ2l0aHViLmNvbS91c2Vycy9qcmV2ZWxzL3JlcG9zIiwiZXZlbnRzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9ldmVudHN7L3ByaXZhY3l9IiwicmVjZWl2ZWRfZXZlbnRzX3VybCI6Imh0dHBzOi8vYXBpLmdpdGh1Yi5jb20vdXNlcnMvanJldmVscy9yZWNlaXZlZF9ldmVudHMiLCJ0eXBlIjoiVXNlciIsInNpdGVfYWRtaW4iOmZhbHNlfX0="
 

test/event_tests.jl

@@ -1,6 +1,6 @@
 include("commit_comment.jl")
 event_request = create_event()
-event_json = JSON.parse(HTTP.payload(event_request, String))
+event_json = JSON.parse(IOBuffer(HTTP.payload(event_request)))
 event = GitHub.event_from_payload!("commit_comment", event_json)
 
 @testset "WebhookEvent" begin

test/ghtype_tests.jl

@@ -12,7 +12,7 @@ function test_show(g::GitHub.GitHubType)
     @test repr(g) == String(take!(tmpio))
 
     tmpio = IOBuffer()
-    showcompact(tmpio, g)
+    show(IOContext(tmpio, :compact => true), g)
 
     @test "$(typeof(g))($(repr(name(g))))" == String(take!(tmpio))
 end
@@ -540,7 +540,7 @@ end
 
     test_show(hook_result)
 end
-  
+
 @testset "Gist" begin
     gist_json = JSON.parse("""
       {
@@ -734,7 +734,7 @@ end
         "updated_at": "2017-07-08T16:18:44"
       }
     """)
-    
+
     app_result = App(app_json)
     @test name(app_result) == Int(app_json["id"])
 end

test/read_only_api_tests.jl

@@ -189,7 +189,7 @@ end
             @test entry["type"] == "blob"
 
             b = blob(github_jl, entry["sha"]; auth=auth)
-            @test contains(String(base64decode(replace(get(b.content),"\n" => ""))), "GitHub.jl")
+            @test occursin("GitHub.jl", String(base64decode(replace(get(b.content),"\n" => ""))))
 
             break
         end