Merge pull request #2 from Sage-Bionetworks/dockerization

perf: make APP_REDIRECT_URL as environment variable
Sage-Bionetworks · Apr 10, 2023 · 30db80a · 30db80a
2 parents a6e1630 + 0bf7dc1
commit 30db80a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/R/mod-synapse-oauth.R b/R/mod-synapse-oauth.R
@@ -18,14 +18,14 @@ authorization_url <- NULL
 #' @param pkgname default R .onLoad() parameter
 .onLoad <- function(libname, pkgname) {
   synapse <<- reticulate::import("synapseclient", delay_load = TRUE)
-  if (!interactive()) {
+  #if (!interactive()) {
     setup_global_oauth_vars(
-      app_url = get_golem_config("app_url"),
+      app_url = Sys.getenv("APP_REDIRECT_URL"),
       client_name = Sys.getenv("client_name"),
       client_id = Sys.getenv("client_id"),
       client_secret = Sys.getenv("client_secret")
     )
-  }
+  #}
 }
 
 #' @title Synapse Oauth Module