set a default on the redirectUrl agument, and if it is not provided d…

…on't append the query string param
mitodl · Feb 5, 2024 · f4f9234 · f4f9234
1 parent 6c8f042
commit f4f9234
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -12,15 +12,17 @@
 export function initLoginButton(
   containerId,
   baseUrl,
-  redirectUrl,
+  redirectUrl = "",
   buttonText = "Login",
   buttonClass = "",
   loggedInTextClass = "",
 ) {
   const container = document.getElementById(containerId)
   const parsedBaseUrl = new URL(baseUrl)
   const apiUrl = `${parsedBaseUrl.origin}/api/v0/users/me?format=json`
-  const loginUrl = `${parsedBaseUrl.origin}/login/ol-oidc/?redirect_url=${redirectUrl}`
+  const redirectUrlParam =
+    redirectUrl !== "" ? `?redirect_url=${redirectUrl}` : ""
+  const loginUrl = `${parsedBaseUrl.origin}/login/ol-oidc/${redirectUrlParam}`
   fetch(apiUrl, {
     method: "GET",
     credentials: "include",