From 215a0a443f489bbe7e40242e4518cffed09b555b Mon Sep 17 00:00:00 2001
From: Darrel O'Pry <darrel.opry@spry-group.com>
Date: Wed, 12 Aug 2020 19:58:59 -0400
Subject: [PATCH] chore: consolidate into single component file.

---
 src/components/OidcContext.svelte   | 185 ++++++++++++++++++----------
 src/components/components.module.js |  14 ++-
 src/components/oidc.js              |  61 ---------
 3 files changed, 131 insertions(+), 129 deletions(-)
 delete mode 100644 src/components/oidc.js

diff --git a/src/components/OidcContext.svelte b/src/components/OidcContext.svelte
index 8b2f259..8a05fc5 100644
--- a/src/components/OidcContext.svelte
+++ b/src/components/OidcContext.svelte
@@ -1,55 +1,108 @@
-<script>
+<script context="module">
+	import { writable } from 'svelte/store';
+    import { getContext } from 'svelte';
     import oidcClient from 'oidc-client';
-    const { UserManager } = oidcClient;
-    import { onMount, onDestroy, setContext, getContext } from 'svelte';
-    import {
-        OIDC_CONTEXT_REDIRECT_URI,
-        OIDC_CONTEXT_CLIENT_PROMISE,
-        OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI,
-        idToken,
-        accessToken,
-        isAuthenticated,
-        isLoading,
-        authError,
-        userInfo,
-    } from './oidc';
+	const { UserManager } = oidcClient;
+	import { onMount, onDestroy, setContext } from 'svelte';
 
-    // props.
-    export let issuer;
-    export let client_id;
-    export let redirect_uri;
-    export let post_logout_redirect_uri;
-    export let metadata = {};
+	/**
+	 * Stores
+	 */
+	export const isLoading = writable(true);
+	export const isAuthenticated = writable(false);
+	export const accessToken = writable('');
+	export const idToken = writable('');
+	export const userInfo = writable({});
+	export const authError = writable(null);
 
-    setContext(OIDC_CONTEXT_REDIRECT_URI, redirect_uri);
-    setContext(OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI, post_logout_redirect_uri);
+	/**
+	 * Context Keys
+	 *
+	 * using an object literal means the keys are guaranteed not to conflict in any circumstance (since an object only has
+	 * referential equality to itself, i.e. {} !== {} whereas "x" === "x"), even when you have multiple different contexts
+	 * operating across many component layers.
+	 */
+	export const OIDC_CONTEXT_CLIENT_PROMISE = {};
+	export const OIDC_CONTEXT_REDIRECT_URI = {};
+	export const OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI = {};
 
-    // getContext doesn't seem to return a value in OnMount, so we'll pass the oidcPromise around by reference.
-    const settings = {
-        authority: issuer,
-        client_id,
-        response_type: 'id_token token',
-        redirect_uri,
-        post_logout_redirect_uri,
-        response_type: 'code',
-        scope: 'openid profile email',
-        automaticSilentRenew: true,
+	/**
+	 * Refresh the accessToken store.
+	 */
+	export async function refreshToken() {
+		const oidc = await getContext(OIDC_CONTEXT_CLIENT_PROMISE);
+		await oidc.signinSilent();
+	}
+
+	/**
+	 * Initiate Register/Login flow.
+	 *
+	 * @param {boolean} preserveRoute - store current location so callback handler will navigate back to it.
+	 * @param {string} callback_url - explicit path to use for the callback.
+	 */
+	export async function login(preserveRoute = true, callback_url = null) {
+		const oidc = await getContext(OIDC_CONTEXT_CLIENT_PROMISE);
+		const redirect_uri = callback_url || getContext(OIDC_CONTEXT_REDIRECT_URI) || window.location.href;
+
+		// try to keep the user on the same page from which they triggered login. If set to false should typically
+		// cause redirect to /.
+		const appState = preserveRoute
+			? {
+					pathname: window.location.pathname,
+					search: window.location.search,
+			  }
+			: {};
+		await oidc.signinRedirect({ redirect_uri, appState });
+	}
+
+	/**
+	 * Log out the current user.
+	 *
+	 * @param {string} logout_url - specify the url to return to after login.
+	 */
+	export async function logout(logout_url = null) {
+		const oidc = await getContext(OIDC_CONTEXT_CLIENT_PROMISE);
+		const returnTo = logout_url || getContext(OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI) ||	window.location.href;
+		oidc.signoutRedirect({ returnTo });
+	}
+</script>
+
+<script>
+	// props.
+	export let issuer;
+	export let client_id;
+	export let redirect_uri;
+	export let post_logout_redirect_uri;
+	export let metadata = {};
+
+	setContext(OIDC_CONTEXT_REDIRECT_URI, redirect_uri);
+	setContext(OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI, post_logout_redirect_uri);
+
+	const settings = {
+		authority: issuer,
+		client_id,
+		// response_type: 'id_token token',
+		redirect_uri,
+		post_logout_redirect_uri,
+		response_type: 'code',
+		scope: 'openid profile email',
+		automaticSilentRenew: true,
         metadata
-    };
+	};
 
-    const userManager = new UserManager(settings);
+	const userManager = new UserManager(settings);
     userManager.events.addUserLoaded(function (user) {
-        isAuthenticated.set(true);
-        accessToken.set(user.access_token);
-        idToken.set(user.id_token);
-        userInfo.set(user.profile);
-    });
+		isAuthenticated.set(true);
+		accessToken.set(user.access_token);
+		idToken.set(user.id_token);
+		userInfo.set(user.profile);
+	});
 
     userManager.events.addUserUnloaded(function () {
-        isAuthenticated.set(false);
-        idToken.set('');
-        accessToken.set('');
-        userInfo.set({});
+		isAuthenticated.set(false);
+		idToken.set('');
+		accessToken.set('');
+		userInfo.set({});
     });
 
     userManager.events.addSilentRenewError(function (e) {
@@ -61,40 +114,40 @@
     setContext(OIDC_CONTEXT_CLIENT_PROMISE, oidcPromise);
 
 
-    async function handleOnMount() {
-        // on run onMount after oidc
+	async function handleOnMount() {
+		// on run onMount after oidc
         const oidc = await oidcPromise;
 
         // Not all browsers support this, please program defensively!
         const params = new URLSearchParams(window.location.search);
 
-        // Check if something went wrong during login redirect
-        // and extract the error message
-        if (params.has('error')) {
-            authError.set(new Error(params.get('error_description')));
-        }
+		// Check if something went wrong during login redirect
+		// and extract the error message
+		if (params.has('error')) {
+			authError.set(new Error(params.get('error_description')));
+		}
 
-        // if code then login success
-        if (params.has('code')) {
-            // handle the callback
-            const response = await oidc.signinCallback();
+		// if code then login success
+		if (params.has('code')) {
+			// handle the callback
+			const response = await oidc.signinCallback();
             let state = (response && response.state) || {}
-            // Can be smart here and redirect to original path instead of root
+			// Can be smart here and redirect to original path instead of root
             const url = state && state.targetUrl ? state.targetUrl : window.location.pathname;
-            state = {  ...state,  isRedirectCallback: true };
+			state = { ...state, isRedirectCallback: true };
 
-            // redirect to the last page we were on when login was configured if it was passed.
+			// redirect to the last page we were on when login was configured if it was passed.
             history.replaceState(state, "", url);
-            // location.href = url;
-            // clear errors on login.
-            authError.set(null);
-        }
-        isLoading.set(false);
-    }
-    async function handleOnDestroy() {}
+			// location.href = url;
+			// clear errors on login.
+			authError.set(null);
+		}
+		isLoading.set(false);
+	}
+	async function handleOnDestroy() {}
 
-    onMount(handleOnMount);
-    onDestroy(handleOnDestroy);
+	onMount(handleOnMount);
+	onDestroy(handleOnDestroy);
 </script>
 
 
diff --git a/src/components/components.module.js b/src/components/components.module.js
index 22d61c8..bcf4147 100644
--- a/src/components/components.module.js
+++ b/src/components/components.module.js
@@ -1,3 +1,13 @@
-export * from './oidc'
-export { default as OidcContext } from './OidcContext.svelte';
+export {
+    default as OidcContext,
+    authError,
+    idToken,
+    accessToken,
+    isAuthenticated,
+    isLoading,
+    login,
+    logout,
+    refreshToken,
+    userInfo,
+} from './OidcContext.svelte';
 
diff --git a/src/components/oidc.js b/src/components/oidc.js
deleted file mode 100644
index e0a5c4e..0000000
--- a/src/components/oidc.js
+++ /dev/null
@@ -1,61 +0,0 @@
-import { writable } from 'svelte/store';
-import { getContext } from 'svelte';
-
-/**
- * Stores
- */
-export const isLoading = writable(true);
-export const isAuthenticated = writable(false);
-export const accessToken = writable('');
-export const idToken = writable('');
-export const userInfo = writable({});
-export const authError = writable(null);
-
-/**
- * Context Keys
- *
- * using an object literal means the keys are guaranteed not to conflict in any circumstance (since an object only has
- * referential equality to itself, i.e. {} !== {} whereas "x" === "x"), even when you have multiple different contexts
- * operating across many component layers.
- */
-export const OIDC_CONTEXT_CLIENT_PROMISE = {};
-export const OIDC_CONTEXT_REDIRECT_URI = {};
-export const OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI = {};
-
-/**
- * Refresh the accessToken store.
- */
-export async function refreshToken() {
-    const oidc = await getContext(OIDC_CONTEXT_CLIENT_PROMISE)
-    const token = await oidc.signinSilent();
-    accessToken.set(token.accessToken);
-    idToken.set(token.idToken);
-}
-
-/**
- * Initiate Register/Login flow.
- *
- * @param {boolean} preserveRoute - store current location so callback handler will navigate back to it.
- * @param {string} callback_url - explicit path to use for the callback.
- */
-export async function login(preserveRoute = true, callback_url = null) {
-    const oidc = await getContext(OIDC_CONTEXT_CLIENT_PROMISE)
-    const redirect_uri =  callback_url || getContext(OIDC_CONTEXT_REDIRECT_URI) || window.location.href;
-
-    // try to keep the user on the same page from which they triggered login. If set to false should typically
-    // cause redirect to /.
-    const appState = (preserveRoute) ? { pathname: window.location.pathname, search: window.location.search } : {}
-    await oidc.signinRedirect({ redirect_uri, appState });
-}
-
-/**
- * Log out the current user.
- *
- * @param {string} logout_url - specify the url to return to after login.
- */
-export async function logout(logout_url = null) {
-    const oidc = await getContext(OIDC_CONTEXT_CLIENT_PROMISE)
-    const returnTo = logout_url || getContext(OIDC_CONTEXT_POST_LOGOUT_REDIRECT_URI) || window.location.href;
-    accessToken.set('');
-    oidc.signoutRedirect({ returnTo });
-}
\ No newline at end of file