Fix a bug, and improve an error message (#24)

JuliaHealth · Jul 13, 2021 · 6c26f7b · andreasnoack · Jul 14, 2021 · DilumAluthge
1 parent 29c63cd
commit 6c26f7b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Project.toml b/Project.toml
@@ -1,7 +1,7 @@
 name = "SMARTAppLaunch"
 uuid = "d2818e16-7729-416e-a683-becc19f3d01b"
 authors = ["Dilum Aluthge", "contributors"]
-version = "1.1.0"
+version = "1.1.1"
 
 [deps]
 HTTP = "cd3eb016-35fb-5094-929b-558a96fad6f3"

diff --git a/src/provider/ehr_launch.jl b/src/provider/ehr_launch.jl
@@ -21,7 +21,7 @@ end
 """
 function provider_ehr_launch(config::ProviderEHRLaunchConfig,
                              uri::URIs.URI;
-                             scope::AbstractString = _default_scope)
+                             kwargs...)
     queryparams = URIs.queryparams(uri)::Dict{String, String}
     return provider_ehr_launch(config, queryparams; kwargs...)
 end
@@ -84,6 +84,12 @@ function provider_ehr_launch(config::ProviderEHRLaunchConfig;
         redirect = false,
     )
     headers = Dict(authorize_response.headers)
+    auth_error_msg = string(
+        "Something when wrong when authenticating to the EHR. ",
+        "One possible explanation is that your `redirect_uri` value does not exactly ",
+        "match any of the `redirect_uri` values that are on file with the EHR.",
+    )
+    haskey(headers, "Location") || @error(auth_error_msg)
     location = headers["Location"]
     location_uri = URIs.URI(location)
     location_queryparams = URIs.queryparams(location_uri)