Update to HTTP v1 and drop support for older versions of HTTP (#7)

* CompatHelper: bump compat for HTTP to 1, (keep existing compat)

* Drop HTTP 0.9

* Fix compatibility with HTTP 1

* Bump Julia compat to 1.6

* Add finalize job

* Allow nightly test failures

---------

Co-authored-by: CompatHelper Julia <[email protected]>
Co-authored-by: David Widmann <[email protected]>
Co-authored-by: Dilum Aluthge <[email protected]>

Author: 4 people

.github/workflows/ci.yml

@@ -8,14 +8,29 @@ on:
       - master
     tags: '*'
 jobs:
+  finalize:
+    needs: [test, docs]
+    if: always()
+    runs-on: ubuntu-latest
+    steps:
+      - run: |
+          echo test: ${{ needs.test.result }}
+          echo docs: ${{ needs.docs.result }}
+      - run: exit 1
+        # Make sure that every line (except for the last line) ends in `||`.
+        # Make sure that the last line does NOT end in `||`.
+        if: |
+          (needs.test.result != 'success') ||
+          (needs.docs.result != 'success')
   test:
     name: Julia ${{ matrix.version }} - ${{ matrix.os }} - ${{ matrix.arch }} - ${{ github.event_name }}
     runs-on: ${{ matrix.os }}
+    continue-on-error: ${{ matrix.version == 'nightly' }}
     strategy:
       fail-fast: false
       matrix:
         version:
-          - '1.5'
+          - '1.6'
           - '1'
           - 'nightly'
         os:

Project.toml

@@ -1,7 +1,7 @@
 name = "SMARTBackendServices"
 uuid = "78af60b6-7677-4c75-8291-bd270d1b4390"
 authors = ["Dilum Aluthge", "contributors"]
-version = "2.0.0"
+version = "2.1.0"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"
@@ -13,12 +13,12 @@ Random = "9a3f8284-a2c9-5f02-9a11-845980a1fd5c"
 URIs = "5c2747f8-b7ea-4ff2-ba2e-563bfd36b1d4"
 
 [compat]
-HTTP = "0.9.3"
+HTTP = "1"
 HealthBase = "1.0.1"
 JSON3 = "1.5.1"
 JWTs = "0.2.4"
 URIs = "1.2"
-julia = "1.5"
+julia = "1.6"
 
 [extras]
 MbedTLS = "739be429-bea8-5141-9913-cc70e7f3736d"

src/backend_services.jl

@@ -34,7 +34,7 @@ function _token_endpoint_wellknown(config::BackendServicesConfig)
         # In principle, it should be possible to omit the header
         # (and servers may ignore it anyway)
         # Ref: https://www.hl7.org/fhir/smart-app-launch/conformance.html#using-well-known
-        headers = ("Accept" => "application/json",),
+        headers = ["Accept" => "application/json"],
         # Old servers might still only support the /metadata endpoint (even though its use for SMART capabilities is deprecated)
         # Hence we do not throw an exception if the request fails but try the /metadata endpoint first
         # Ref: https://hl7.org/fhir/smart-app-launch/1.0.0/conformance/index.html#declaring-support-for-oauth2-endpoints
@@ -66,7 +66,7 @@ function _token_endpoint_metadata(config::BackendServicesConfig)
         joinpath(config.base_url, "metadata");
         # We only support FHIR version R4
         # Ref: https://hl7.org/fhir/R4/versioning.html#mt-version
-        headers = ("Accept" => "application/fhir+json; fhirVersion=4.0"),
+        headers = ["Accept" => "application/fhir+json; fhirVersion=4.0"],
         # We throw our own, hopefully more descriptive, exception if necessary
         status_exception = false,
     )
@@ -141,7 +141,7 @@ function backend_services(config::BackendServicesConfig)
     _response = HTTP.request(
         "POST",
         token_endpoint;
-        headers = ("Content-Type" => "application/x-www-form-urlencoded",),
+        headers = ["Content-Type" => "application/x-www-form-urlencoded"],
         body = URIs.escapeuri(body_params),
     )