Merge pull request #185 from plotly/dev

Release 1.2.0

Author: etpinard

.circleci/config.yml

@@ -57,3 +57,4 @@ workflows:
   build:
     jobs:
       - "test"
+    when: false  # disable this workflow until Percy tests are functional again

.github/workflows/jl_test.yml

@@ -0,0 +1,20 @@
+name: Run Julia tests
+
+on: [push, pull_request]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        jl_version: ["1.6", "1.8"]
+    steps:
+      - uses: actions/checkout@v2
+      - uses: julia-actions/setup-julia@v1
+        with:
+          version: ${{ matrix.jl_version }}
+      - uses: julia-actions/julia-buildpkg@v1
+      - uses: julia-actions/julia-runtest@v1
+        with:
+          annotate: true

CompatHelper.yml

@@ -1,7 +1,7 @@
 name = "Dash"
 uuid = "1b08a953-4be3-4667-9a23-3db579824955"
 authors = ["Chris Parmer <chris@plotly.com>", "Alexandr Romanenko <waralex@gmail.com>"]
-version = "1.1.2"
+version = "1.2.0"
 
 [deps]
 Base64 = "2a0f44e3-6c83-55bd-87e4-b1978d98bd5f"
@@ -30,14 +30,14 @@ DashCoreComponents = "2.0.0"
 DashHtmlComponents = "2.0.0"
 DashTable = "5.0.0"
 DataStructures = "0.17, 0.18"
-HTTP = "0.8.10, 0.9"
+HTTP = "1"
 JSON = "0.21"
 JSON3 = "1.9"
 MD5 = "0.2"
 PlotlyBase = "0.8.5, 0.8.6"
-YAML = "0.4.7"
-julia = "1.3"
 Requires = "1.3"
+YAML = "0.4.7"
+julia = "1.6"
 
 [extras]
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

Project.toml

@@ -1,5 +1,6 @@
 # Dash for Julia
 
+[![Juila tests](https://github.com/plotly/Dash.jl/actions/workflows/jl_test.yml/badge.svg)](https://github.com/plotly/Dash.jl/actions/workflows/jl_test.yml)
 [![CircleCI](https://circleci.com/gh/plotly/Dash.jl/tree/master.svg?style=svg)](https://circleci.com/gh/plotly/Dash.jl/tree/master)
 [![GitHub](https://img.shields.io/github/license/plotly/dashR.svg?color=dark-green)](https://github.com/plotly/Dash.jl/blob/master/LICENSE)
 [![GitHub commit activity](https://img.shields.io/github/commit-activity/y/plotly/Dash.jl.svg?color=dark-green)](https://github.com/plotly/Dash.jl/graphs/contributors)

README.md

@@ -1,10 +1,9 @@
 module HttpHelpers
 
-export state_handler, exception_handling_handler, compress_handler, Route, Router, add_route!
+export state_handler, exception_handling_handler, compress_handler, Route, Router, add_route!, handle
 
 import HTTP, CodecZlib
-
 include("handlers.jl")
 include("router.jl")
 
-end
+end

src/HttpHelpers/HttpHelpers.jl

@@ -1,7 +1,28 @@
+struct RequestHandlerFunction <: HTTP.Handler
+    func::Function # func(req)
+end
+(x::RequestHandlerFunction)(args...) = x.func(args...)
+
+function handle(h::RequestHandlerFunction, request::HTTP.Request, args...)
+    h(request, args...)
+end
+
+function handle(handler::Function, request::HTTP.Request, args...)
+    handler(request, args)
+end
+
+function handle(h::RequestHandlerFunction, request::HTTP.Request, state, args...)
+    h(request, state, args...)
+end
+
+function handle(handler::Function, request::HTTP.Request, state, args...)
+    handler(request, state, args)
+end
+
 function state_handler(base_handler, state)
-    return HTTP.RequestHandlerFunction(
+    return RequestHandlerFunction(
         function(request::HTTP.Request, args...)
-            response = HTTP.handle(base_handler, request, state, args...)
+            response = handle(base_handler, request, state, args...)
             if response.status == 200
                 HTTP.defaultheader!(response, "Content-Type" => HTTP.sniff(response.body))
                 HTTP.defaultheader!(response, "Content-Length" => string(sizeof(response.body)))
@@ -11,7 +32,7 @@ function state_handler(base_handler, state)
     )
 end
 
-state_handler(base_handler::Function, state) = state_handler(HTTP.RequestHandlerFunction(base_handler), state)
+state_handler(base_handler::Function, state) = state_handler(RequestHandlerFunction(base_handler), state)
 
 function check_mime(message::HTTP.Message, mime_list)
     !HTTP.hasheader(message, "Content-Type") && return false
@@ -22,9 +43,9 @@ end
 
 const default_compress_mimes = ["text/plain", "text/html", "text/css", "text/xml", "application/json", "application/javascript", "application/css"]
 function compress_handler(base_handler; mime_types::Vector{String} = default_compress_mimes, compress_min_size = 500)
-    return HTTP.RequestHandlerFunction(
+    return RequestHandlerFunction(
         function(request::HTTP.Request, args...)
-            response = HTTP.handle(base_handler, request, args...)
+            response = handle(base_handler, request, args...)
             if response.status == 200 && sizeof(response.body) >= compress_min_size &&
             occursin("gzip", HTTP.header(request, "Accept-Encoding", "")) && check_mime(response, mime_types)
                 HTTP.setheader(response, "Content-Encoding" => "gzip")
@@ -38,14 +59,14 @@ function compress_handler(base_handler; mime_types::Vector{String} = default_com
 end
 
 function compress_handler(base_handler::Function; mime_types::Vector{String} = default_compress_mimes, compress_min_size = 500)
-    return compress_handler(HTTP.RequestHandlerFunction(base_handler), mime_types = mime_types, compress_min_size = compress_min_size)
+    return compress_handler(RequestHandlerFunction(base_handler), mime_types = mime_types, compress_min_size = compress_min_size)
 end
 
 function exception_handling_handler(ex_handling_func, base_handler)
-    return HTTP.RequestHandlerFunction(
+    return RequestHandlerFunction(
         function(request::HTTP.Request, args...)
             try
-                return HTTP.handle(base_handler, request, args...)
+                return handle(base_handler, request, args...)
             catch e
                 return ex_handling_func(e)
             end
@@ -55,12 +76,12 @@ function exception_handling_handler(ex_handling_func, base_handler)
 end
 
 exception_handling_handler(ex_handling_func, base_handler::Function) =
-         exception_handling_handler(ex_handling_func, HTTP.RequestHandlerFunction(base_handler))
+         exception_handling_handler(ex_handling_func, RequestHandlerFunction(base_handler))
 
 function request_logging_handler(base_handler; exclude = Regex[])
-    return HTTP.RequestHandlerFunction(
+    return RequestHandlerFunction(
         function(request::HTTP.Request, args...)
-            response = HTTP.handle(base_handler, request, args...)
+            response = handle(base_handler, request, args...)
 
             return response
         end

src/HttpHelpers/handlers.jl

@@ -124,7 +124,7 @@ end
 add_route!(handler::Function, router::Router, method, url) = add_route!(router, Route(handler, method, url))
 add_route!(handler::Function, router::Router, url) = add_route!(handler, router, nothing, url)
 
-function HTTP.handle(router::Router, request::HTTP.Request, args...)
+function handle(router::Router, request::HTTP.Request, args...)
     path = HTTP.URI(request.target).path
     return handle(router.routes, path, request, args...)
 end

src/HttpHelpers/router.jl

@@ -147,7 +147,7 @@ function make_handler(app::DashApp, registry::ResourcesRegistry; check_layout =
 
     compile_request = HTTP.Request("GET", prefix)
     HTTP.setheader(compile_request, "Accept-Encoding" => "gzip")
-    HTTP.handle(handler, compile_request) #For handler precompilation
+    handle(handler, compile_request) #For handler precompilation
 
     get_devsetting(app, :hot_reload) && start_reload_poll(state)
 

src/handler/make_handler.jl

@@ -52,7 +52,7 @@ function run_server(app::DashApp,
     start_server = () -> begin
         handler = make_handler(app);
         server = Sockets.listen(get_inetaddr(host, port))
-        task = @async HTTP.serve(handler, host, port; server = server, verbose = true)
+        task = @async HTTP.serve(handler, host, port; server = server)
         return (server, task)
     end