Throw a better exception (instead of logging an error and throwing a …

…less informative exception) (#26)
JuliaHealth · Jul 15, 2021 · fc95e1b · fc95e1b · DilumAluthge · Jul 15, 2021
1 parent 6c26f7b
commit fc95e1b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 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.1"
+version = "1.1.2"
 
 [deps]
 HTTP = "cd3eb016-35fb-5094-929b-558a96fad6f3"

diff --git a/src/provider/ehr_launch.jl b/src/provider/ehr_launch.jl
@@ -84,12 +84,14 @@ 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)
+    let 
+        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") || throw(ErrorException(auth_error_msg))
+    end
     location = headers["Location"]
     location_uri = URIs.URI(location)
     location_queryparams = URIs.queryparams(location_uri)