diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8b6763f..da52507 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,4 @@
-## Cockpit File Sharing 3.2.3-1
+## Cockpit File Sharing 3.2.4-1
 
-* Fixed issue saving NFS exports file when /etc/exports.d DNE
\ No newline at end of file
+* Add settings menu for configuring smb.conf and exports file locations
+* Fix parsing "valid users" field for Samba shares
\ No newline at end of file
diff --git a/file-sharing/src/components/Config.vue b/file-sharing/src/components/Config.vue
new file mode 100644
index 0000000..6c0c3b6
--- /dev/null
+++ b/file-sharing/src/components/Config.vue
@@ -0,0 +1,180 @@
+<!--
+Copyright (C) 2022 Josh Boudreau <jboudreau@45drives.com>
+
+This file is part of Cockpit File Sharing.
+
+Cockpit File Sharing is free software: you can redistribute it and/or modify it under the terms
+of the GNU General Public License as published by the Free Software Foundation, either version 3
+of the License, or (at your option) any later version.
+
+Cockpit File Sharing is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY;
+without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License along with Cockpit File Sharing.
+If not, see <https://www.gnu.org/licenses/>. 
+-->
+
+<template>
+	<button
+		type="button"
+		@click="show = true"
+	>
+		<CogIcon class="size-icon icon-default" />
+	</button>
+	<ModalPopup
+		:showModal="show"
+		headerText="File Sharing Settings"
+		cancelText="Close"
+		@apply="writeConfig"
+		@cancel="hide"
+		:disableContinue="noChanges"
+	>
+		<div class="flex flex-col items-stretch gap-content">
+			<div class="text-header">Samba</div>
+			<div>
+				<label class="text-label block">
+					Configuration Path
+				</label>
+				<input
+					v-model="configTmp.samba.confPath"
+					type="text"
+					class="input-textlike w-full block"
+					placeholder="default: /etc/samba/smb.conf"
+				/>
+			</div>
+			<div class="text-header">NFS</div>
+			<div>
+				<label class="text-label block">
+					Configuration Path
+				</label>
+				<input
+					v-model="configTmp.nfs.confPath"
+					type="text"
+					class="input-textlike w-full block"
+					placeholder="default: /etc/exports.d/cockpit-file-sharing.exports"
+				/>
+			</div>
+		</div>
+	</ModalPopup>
+</template>
+
+<script lang="ts">
+import { reactive, ref, defineComponent, watch, computed } from 'vue';
+import { CogIcon } from '@heroicons/vue/solid';
+import ModalPopup from './ModalPopup.vue';
+
+declare global {
+	var cockpit: any = {};
+}
+
+interface FileSharingConfig {
+	/**
+	 * Samba-specific settings
+	 */
+	samba: {
+		/**
+		 * Path to smb.conf
+		 */
+		confPath: string;
+	}
+	/**
+	 * NFS-specific settings
+	 */
+	nfs: {
+		/**
+		 * Path to cockpit-file-sharing.exports or equivalent
+		 */
+		confPath: string;
+	}
+}
+
+function deepCopy<T>(obj: T): T {
+	return JSON.parse(JSON.stringify(obj));
+}
+
+const assignConfig = (target: any, source: FileSharingConfig, defaults: FileSharingConfig): FileSharingConfig => {
+	Object.assign(target, {
+		samba: {
+			confPath: source?.samba?.confPath || defaults.samba.confPath,
+		},
+		nfs: {
+			confPath: source?.nfs?.confPath || defaults.nfs.confPath,
+		}
+	});
+	return target as FileSharingConfig;
+}
+
+const configPath = "/etc/cockpit-file-sharing.conf.json";
+
+const configFile = cockpit.file(configPath, {
+	superuser: 'try',
+	syntax: {
+		parse: (str: string) => JSON.parse(str),
+		stringify: (obj: any) => JSON.stringify(obj, null, 2),
+	}
+});
+
+const configDefaults: FileSharingConfig = {
+	samba: {
+		confPath: "/etc/samba/smb.conf",
+	},
+	nfs: {
+		confPath: "/etc/exports.d/cockpit-file-sharing.exports"
+	}
+};
+
+const config: FileSharingConfig = reactive(deepCopy(configDefaults));
+
+const loadConfig = (onDiskConfig: FileSharingConfig | null) => {
+	console.log(onDiskConfig);
+	if (onDiskConfig === null) {
+		console.log('file DNE');
+	} else {
+		console.log('has content');
+		assignConfig(config, onDiskConfig, configDefaults);
+	}
+}
+
+configFile.watch(loadConfig);
+
+export function useConfig() {
+	return config;
+}
+
+export default defineComponent({
+	setup() {
+		const show = ref(false);
+		const noChanges = computed(() => configTmp.nfs.confPath === config.nfs.confPath && configTmp.samba.confPath === config.samba.confPath);
+
+		const configTmp: FileSharingConfig = reactive(deepCopy(configDefaults));
+
+		const writeConfig = () => {
+			assignConfig(config, configTmp, configDefaults);
+			configFile.replace(config);
+		}
+
+		const hide = () => {
+			show.value = false;
+			configFile.read(); // trigger watch to reset fields
+		}
+
+		watch(config, () => {
+			assignConfig(configTmp, config, configDefaults);
+		});
+
+		return {
+			show,
+			noChanges,
+			configTmp,
+			hide,
+			writeConfig,
+			config,
+		}
+	},
+	components: {
+		ModalPopup,
+		CogIcon,
+	}
+})
+</script>
diff --git a/file-sharing/src/components/FilePermissions.vue b/file-sharing/src/components/FilePermissions.vue
index 7375486..a585ee7 100644
--- a/file-sharing/src/components/FilePermissions.vue
+++ b/file-sharing/src/components/FilePermissions.vue
@@ -55,7 +55,7 @@ If not, see <https://www.gnu.org/licenses/>.
 </template>
 
 <script>
-import { watch, ref, inject } from 'vue';
+import { watch, ref, inject, onMounted } from 'vue';
 import ModalPopup from "./ModalPopup.vue";
 import FileModeMatrix from "./FileModeMatrix.vue";
 import { useSpawn, errorString, canonicalPath } from "@45drives/cockpit-helpers";
@@ -118,9 +118,11 @@ export default {
 			emit('hide');
 		}
 
-		watch(() => props.show, (show, lastShow) => {
-			if (show || lastShow === undefined) getPermissions();
-		}, { immediate: true });
+		onMounted(() => {
+			watch(() => props.show, (show, lastShow) => {
+				if (show) getPermissions();
+			}, { immediate: true });
+		});
 
 		return {
 			mode,
diff --git a/file-sharing/src/components/HoustonHeader.vue b/file-sharing/src/components/HoustonHeader.vue
index 8685b2b..322d9c9 100644
--- a/file-sharing/src/components/HoustonHeader.vue
+++ b/file-sharing/src/components/HoustonHeader.vue
@@ -33,7 +33,10 @@ If not, see <https://www.gnu.org/licenses/>.
 				</h1>
 			</div>
 			<slot />
-			<LoadingSpinner v-if="showSpinner" class="size-icon self-center" />
+			<LoadingSpinner
+				v-if="showSpinner"
+				class="size-icon self-center"
+			/>
 		</div>
 		<h1
 			class="text-red-800 dark:text-white text-base sm:text-2xl cursor-pointer grow-0 text-center"
@@ -41,19 +44,30 @@ If not, see <https://www.gnu.org/licenses/>.
 		>{{ moduleName }}</h1>
 		<div class="flex basis-32 justify-end items-center grow shrink-0 gap-buttons">
 			<div
-				:class="[infoButtonInHeader ? '' : 'md:fixed md:right-2 md:bottom-2 md:z-50 flex flex-row items-stretch']"
-			>
-				<button @click="showInfo = true">
-					<QuestionMarkCircleIcon class="size-icon icon-default" />
-				</button>
+				:class="[infoButtonInHeader ? '' : 'md:fixed md:right-2 md:bottom-2 md:z-50 flex flex-row items-stretch']">
+				<div class="inline-button-group-row">
+					<Config />
+					<button @click="showInfo = true">
+						<QuestionMarkCircleIcon class="size-icon icon-default" />
+					</button>
+				</div>
 				<div
 					class="overflow-y-auto"
 					:style="{ 'scrollbar-gutter': infoNudgeScrollbar ? 'stable' : 'auto' }"
 				></div>
 			</div>
-			<button @click="darkMode = !darkMode" @click.right.prevent="vape">
-				<SunIcon v-if="darkMode" class="size-icon-lg icon-default" />
-				<MoonIcon v-else class="size-icon-lg icon-default" />
+			<button
+				@click="darkMode = !darkMode"
+				@click.right.prevent="vape"
+			>
+				<SunIcon
+					v-if="darkMode"
+					class="size-icon-lg icon-default"
+				/>
+				<MoonIcon
+					v-else
+					class="size-icon-lg icon-default"
+				/>
 			</button>
 		</div>
 	</div>
@@ -73,8 +87,16 @@ If not, see <https://www.gnu.org/licenses/>.
 					target="_blank"
 				>45Drives</a> for Houston UI (Cockpit)
 			</span>
-			<a class="text-link" :href="sourceURL" target="_blank">Source Code</a>
-			<a class="text-link" :href="issuesURL" target="_blank">Issue Tracker</a>
+			<a
+				class="text-link"
+				:href="sourceURL"
+				target="_blank"
+			>Source Code</a>
+			<a
+				class="text-link"
+				:href="issuesURL"
+				target="_blank"
+			>Issue Tracker</a>
 		</div>
 	</ModalPopup>
 </template>
@@ -87,6 +109,7 @@ import { SunIcon, MoonIcon, QuestionMarkCircleIcon } from "@heroicons/vue/solid"
 import { ref, watch, inject } from "vue";
 import LoadingSpinner from "./LoadingSpinner.vue";
 import ModalPopup from './ModalPopup.vue';
+import Config from "./Config.vue";
 
 export default {
 	props: {
@@ -167,6 +190,7 @@ export default {
 		LoadingSpinner,
 		ModalPopup,
 		QuestionMarkCircleIcon,
+		Config,
 	}
 };
 </script>
diff --git a/file-sharing/src/components/NfsManager.vue b/file-sharing/src/components/NfsManager.vue
index 992c473..09addc8 100644
--- a/file-sharing/src/components/NfsManager.vue
+++ b/file-sharing/src/components/NfsManager.vue
@@ -16,27 +16,47 @@ If not, see <https://www.gnu.org/licenses/>.
 -->
 
 <template>
-	<div class="centered-column p-well space-y-well" :class="{'cursor-wait': processing}">
+	<div
+		class="centered-column p-well space-y-well"
+		:class="{ 'cursor-wait': processing }"
+	>
 		<div class="card">
 			<div class="card-header flex flex-row items-center gap-2">
 				<div class="text-header">Shares</div>
-				<LoadingSpinner v-if="processing" class="size-icon" />
+				<LoadingSpinner
+					v-if="processing"
+					class="size-icon"
+				/>
 			</div>
-			<div class="overflow-hidden"
-				:style="{ 'max-height': showAddShare ? '1500px' : '0', transition: showAddShare ? 'max-height 0.5s ease-in' : 'max-height 0.5s ease-out' }">
+			<div
+				class="overflow-hidden"
+				:style="{ 'max-height': showAddShare ? '1500px' : '0', transition: showAddShare ? 'max-height 0.5s ease-in' : 'max-height 0.5s ease-out' }"
+			>
 				<div class="card-body">
-					<NfsShareEditor @update-share="addShare" @hide="showAddShare = false" :users="users"
-						:groups="groups" />
+					<NfsShareEditor
+						@update-share="addShare"
+						@hide="showAddShare = false"
+						:users="users"
+						:groups="groups"
+					/>
 				</div>
 			</div>
 			<div :class="[showAddShare ? '' : '!border-t-0', 'card-body p-0 sm:p-6']">
 				<div class="sm:shadow sm:rounded-lg sm:border sm:border-default overflow-hidden">
-					<Table emptyText="No shares. Click '+' to add one." shrinkHeight noScroll noHeader
-						class="!border-none !shadow-none">
+					<Table
+						emptyText="No shares. Click '+' to add one."
+						shrinkHeight
+						noScroll
+						noHeader
+						class="!border-none !shadow-none"
+					>
 						<template #thead>
 							<tr>
 								<th scope="col">Path</th>
-								<th scope="col" class="flex flex-row justify-end">
+								<th
+									scope="col"
+									class="flex flex-row justify-end"
+								>
 									<span class="sr-only">Edit/Delete</span>
 									<button @click="showAddShare = !showAddShare">
 										<PlusIcon class="size-icon icon-default" />
@@ -45,9 +65,16 @@ If not, see <https://www.gnu.org/licenses/>.
 							</tr>
 						</template>
 						<template #tbody>
-							<NfsShare v-for="(share, index) in shares" :key="index" :share="shares[index]"
-								:index="index" @delete-share="deleteShare" @update-share="updateShare" :users="users"
-								:groups="groups" />
+							<NfsShare
+								v-for="(share, index) in shares"
+								:key="index"
+								:share="shares[index]"
+								:index="index"
+								@delete-share="deleteShare"
+								@update-share="updateShare"
+								:users="users"
+								:groups="groups"
+							/>
 						</template>
 					</Table>
 				</div>
@@ -56,18 +83,37 @@ If not, see <https://www.gnu.org/licenses/>.
 		<div class="card">
 			<div class="card-header flex flex-row space-x-2 items-center">
 				<div class="text-header">Import/Export Config</div>
-				<LoadingSpinner v-if="processing" class="size-icon" />
+				<LoadingSpinner
+					v-if="processing"
+					class="size-icon"
+				/>
 			</div>
 			<div class="card-body button-group-row">
-				<input @change="importConfig" type="file" id="file-upload" hidden />
-				<button @click="uploadConfig" class="btn btn-primary">Import</button>
-				<button @click="exportConfig" class="btn btn-primary">Export</button>
+				<input
+					@change="importConfig"
+					type="file"
+					id="file-upload"
+					hidden
+				/>
+				<button
+					@click="uploadConfig"
+					class="btn btn-primary"
+				>Import</button>
+				<button
+					@click="exportConfig"
+					class="btn btn-primary"
+				>Export</button>
 			</div>
 		</div>
 	</div>
-	<ModalPopup :showModal="confirmationModal.showModal" @apply="confirmationModal.applyCallback"
-		@cancel="confirmationModal.cancelCallback" applyDangerous applyText="Yes"
-		:headerText="confirmationModal.headerText">
+	<ModalPopup
+		:showModal="confirmationModal.showModal"
+		@apply="confirmationModal.applyCallback"
+		@cancel="confirmationModal.cancelCallback"
+		applyDangerous
+		applyText="Yes"
+		:headerText="confirmationModal.headerText"
+	>
 		<template #icon>
 			<ExclamationCircleIcon class="size-icon-xl icon-danger shrink-0" />
 		</template>
@@ -80,22 +126,25 @@ import NfsShare from "./NfsShare.vue";
 import NfsShareEditor from "./NfsShareEditor.vue";
 import { PlusIcon, XCircleIcon, ExclamationCircleIcon } from "@heroicons/vue/solid";
 import LoadingSpinner from "./LoadingSpinner.vue";
-import { ref, reactive, inject, onBeforeUnmount } from "vue";
+import { ref, reactive, inject, onBeforeUnmount, watch } from "vue";
 import { NfsExportSyntax } from "@45drives/cockpit-syntaxes";
 import { useSpawn, errorString, errorStringHTML, fileDownload } from "@45drives/cockpit-helpers";
 import Table from "./Table.vue";
 import { notificationsInjectionKey, usersInjectionKey, groupsInjectionKey } from "../keys";
 import ModalPopup from "./ModalPopup.vue";
+import { useConfig } from "./Config.vue";
 
 export default {
 	setup(props) {
+		const config = useConfig();
 		const shares = ref([]);
 		const processing = ref(0);
 		const showAddShare = ref(false);
-		const exportsFile = reactive(cockpit.file("/etc/exports.d/cockpit-file-sharing.exports", { superuser: 'try', syntax: NfsExportSyntax }));
 		const users = inject(usersInjectionKey);
 		const groups = inject(groupsInjectionKey);
 		const notifications = inject(notificationsInjectionKey);
+		let exportsFile = null;
+		let exportsFileWatchHandle = null;
 
 		const confirmationModal = reactive({
 			showModal: false,
@@ -118,11 +167,10 @@ export default {
 			cancelCallback: () => { },
 		});
 
-		const loadShares = async () => {
+		const loadShares = (sharesOnDisk) => {
 			processing.value++;
 			try {
-				shares.value = await exportsFile.read() ?? [];
-				shares.value.sort((a, b) => a.path.localeCompare(b.path));
+				shares.value = sharesOnDisk.sort((a, b) => a.path.localeCompare(b.path)) ?? [];
 			} catch (error) {
 				notifications.value.constructNotification("Failed to load share configuration", errorStringHTML(error), 'error', 0);
 			} finally {
@@ -130,18 +178,8 @@ export default {
 			}
 		};
 
-		const init = async () => {
-			const procs = [];
-			procs.push(loadShares());
-			for (let proc of procs)
-				await proc;
-		};
-
-		init();
-
 		const writeExportsFile = async () => {
 			try {
-				await useSpawn(['mkdir', '-p', '/etc/exports.d'], { superuser: 'try' }).promise();
 				await exportsFile.replace(shares.value);
 			} catch (error) {
 				error.message = `Failed to write exports file: ${errorString(error.message)}`;
@@ -160,13 +198,14 @@ export default {
 
 		const updateShare = async (share, newShare) => {
 			const oldShare = {};
+			const rollback = [...shares.value];
 			processing.value++;
 			try {
 				if (share) {
 					Object.assign(oldShare, share);
 					Object.assign(share, newShare);
 				} else {
-					shares.value.push({ ...newShare });
+					shares.value = [...shares.value, { ...newShare }];
 				}
 				await writeExportsFile();
 				await validateExportsFile();
@@ -176,7 +215,7 @@ export default {
 				if (share) {
 					Object.assign(share, oldShare);
 				} else {
-					shares.value.pop();
+					shares.value = rollback;
 				}
 				try {
 					await writeExportsFile();
@@ -184,7 +223,6 @@ export default {
 					notifications.value.constructNotification("Failed to revert exports file", errorStringHTML(error), 'error');
 				}
 			} finally {
-				shares.value.sort((a, b) => a.path.localeCompare(b.path));
 				processing.value--;
 			}
 		}
@@ -202,7 +240,6 @@ export default {
 				notifications.value.constructNotification(`Successfully deleted share`, "", 'success');
 			} catch (error) {
 				notifications.value.constructNotification("Failed to delete share", errorStringHTML(error), 'error');
-				loadShares();
 			}
 		}
 
@@ -226,7 +263,6 @@ export default {
 					notifications.value.constructNotification("Failed to import config", errorStringHTML(error), 'error');
 					shares.value = oldShares;
 				}
-				shares.value.sort((a, b) => a.path.localeCompare(b.path));
 				processing.value--;
 			}
 			reader.onerror = (event) => {
@@ -243,6 +279,12 @@ export default {
 			fileDownload(exportsFile.path, filename);
 		}
 
+		watch(() => config.nfs.confPath, () => {
+			exportsFileWatchHandle?.remove();
+			exportsFile = cockpit.file(config.nfs.confPath, { superuser: 'try', syntax: NfsExportSyntax });
+			exportsFileWatchHandle = exportsFile.watch(loadShares);
+		}, { immediate: true });
+
 		return {
 			shares,
 			processing,
diff --git a/file-sharing/src/components/SambaManager.vue b/file-sharing/src/components/SambaManager.vue
index ff4bdeb..11ba054 100644
--- a/file-sharing/src/components/SambaManager.vue
+++ b/file-sharing/src/components/SambaManager.vue
@@ -65,11 +65,11 @@ If not, see <https://www.gnu.org/licenses/>.
 						@click="importSmbConf"
 						class="btn btn-secondary"
 					>
-						Import configuration from <span class="text-sm font-mono">/etc/samba/smb.conf</span>
+						Import configuration from <span class="text-sm font-mono">{{ config.samba.confPath }}</span>
 					</button>
 					<InfoTip above>
 						File Sharing uses Samba's net registry to configure shares. Click this button to import
-						configuration from <span class="text-sm font-mono">/etc/samba/smb.conf</span> into the net
+						configuration from <span class="text-sm font-mono">{{ config.samba.confPath }}</span> into the net
 						registry for management.
 					</InfoTip>
 				</div>
@@ -107,9 +107,11 @@ import LoadingSpinner from "./LoadingSpinner.vue";
 import { notificationsInjectionKey, usersInjectionKey, groupsInjectionKey } from "../keys";
 import ModalPopup from "./ModalPopup.vue";
 import InfoTip from "./InfoTip.vue";
+import { useConfig } from "./Config.vue";
 
 export default {
 	setup(props, ctx) {
+		const config = useConfig();
 		const shares = ref([]);
 		const globalConfig = reactive({ advancedSettings: [] });
 		const users = inject(usersInjectionKey);
@@ -204,13 +206,13 @@ export default {
 		const checkConf = async () => {
 			processing.value++;
 			try {
-				const smbConfFile = cockpit.file("/etc/samba/smb.conf", { superuser: 'try' });
+				const smbConfFile = cockpit.file(config.samba.confPath, { superuser: 'try' });
 				const smbConf = await smbConfFile.read();
 				const globalSectionText = smbConf.match(/^\s*\[ ?global ?\].*$(?:\n^(?!\s*\[).*$)*/mi)?.[0];
 				if (!globalSectionText || !/^[\t ]*include[\t ]*=[\t ]*registry/m.test(globalSectionText)) {
 					notifications.value.constructNotification(
 						"Samba is Misconfigured",
-						"`include = registry` is missing from the global section of /etc/samba/smb.conf, which is required for File Sharing to manage shares.",
+						`'include = registry' is missing from the global section of ${config.samba.confPath}, which is required for File Sharing to manage shares.`,
 						'error'
 					).addAction("Fix now", async () => {
 						processing.value++;
@@ -228,12 +230,14 @@ export default {
 						}
 					});
 				}
-				// fix end of line comment from previous version of file sharing
-				smbConfFile.modify(content =>
-					content.replace('include = registry # inserted by cockpit-file-sharing', '# inclusion of net registry, inserted by cockpit-file-sharing:\n\tinclude = registry')
-				);
+				if (/include = registry # inserted by cockpit-file-sharing/.test(smbConf)) {
+					// fix end of line comment from previous version of file sharing
+					smbConfFile.modify(content =>
+						content.replace('include = registry # inserted by cockpit-file-sharing', '# inclusion of net registry, inserted by cockpit-file-sharing:\n\tinclude = registry')
+					);
+				}
 			} catch (error) {
-				notifications.value.constructNotification("Failed to validate /etc/samba/smb.conf: ", errorStringHTML(error), 'error');
+				notifications.value.constructNotification(`Failed to validate ${config.samba.confPath}: `, errorStringHTML(error), 'error');
 			} finally {
 				processing.value--;
 			}
@@ -277,7 +281,6 @@ export default {
 			try {
 				const procs = [];
 				procs.push(parseNetConf().then(() => shares.value.sort((a, b) => a.name.localeCompare(b.name))));
-				procs.push(checkConf());
 				procs.push(getCtdbHosts());
 				procs.push(getCephLayoutPools());
 				await Promise.all(procs);
@@ -314,7 +317,7 @@ export default {
 		}
 
 		const importSmbConf = async () => {
-			const testContent = (await useSpawn(['net', 'conf', 'import', '-T', '/etc/samba/smb.conf'], { superuser: 'try' }).promise()).stdout;
+			const testContent = (await useSpawn(['net', 'conf', 'import', '-T', config.samba.confPath], { superuser: 'try' }).promise()).stdout;
 			if (!await confirmationModal.ask(
 				"This will permanently overwrite current configuration",
 				"New configuration content:\n" +
@@ -329,12 +332,12 @@ export default {
 				return;
 			try {
 				processing.value++;
-				const smbConfFile = cockpit.file("/etc/samba/smb.conf", { superuser: 'try' });
+				const smbConfFile = cockpit.file(config.samba.confPath, { superuser: 'try' });
 				const smbConf = await smbConfFile.read();
 				smbConfFile.close();
 				await importConfig(smbConf);
 			} catch (error) {
-				notifications.value.constructNotification("Failed to read /etc/samba/smb.conf", errorStringHTML(error), 'error');
+				notifications.value.constructNotification(`Failed to read ${config.samba.confPath}`, errorStringHTML(error), 'error');
 				return;
 			} finally {
 				processing.value--;
@@ -342,9 +345,9 @@ export default {
 			await new Promise(resolve => setTimeout(resolve, 300)); // temporary hack until modals are fixed
 			if (await confirmationModal.ask(
 				'Replace smb.conf to avoid conflicts?',
-				'Your original <span class="text-sm font-mono">/etc/samba/smb.conf</span> ' +
+				`Your original <span class="text-sm font-mono">${config.samba.confPath}</span> ` +
 				'will be backed up ' +
-				'to <span class="text-sm font-mono">/etc/samba/smb.conf.bak</span> and replaced with\n' +
+				`to <span class="text-sm font-mono">${config.samba.confPath}.bak</span> and replaced with\n` +
 				'<span class="text-sm font-mono whitespace-pre">\n' +
 				'[global]\n' +
 				'	include = registry\n' +
@@ -353,10 +356,10 @@ export default {
 				false
 			)) {
 				try {
-					const backupDescription = (await useSpawn(['cp', '-v', '--backup=numbered', '/etc/samba/smb.conf', '/etc/samba/smb.conf.bak'], { superuser: 'try' }).promise()).stdout;
+					const backupDescription = (await useSpawn(['cp', '-v', '--backup=numbered', config.samba.confPath, `${config.samba.confPath}.bak`], { superuser: 'try' }).promise()).stdout;
 					notifications.value.constructNotification("Backed up original smb.conf", backupDescription.trim(), 'info');
 					try {
-						await cockpit.file('/etc/samba/smb.conf', { superuser: 'try' }).replace(
+						await cockpit.file(config.samba.confPath, { superuser: 'try' }).replace(
 							'# this config was generated by cockpit-file-sharing after importing smb.conf\n' +
 							`# original smb.conf location:\n` +
 							backupDescription.replace(/^/m, '# ') +
@@ -364,11 +367,11 @@ export default {
 							'	include = registry\n');
 						await useSpawn(['smbcontrol', 'all', 'reload-config'], { superuser: 'try' }).promise();
 					} catch (error) {
-						notifications.value.constructNotification("Failed to replace contents of /etc/samba/smb.conf", errorStringHTML(error), 'error');
+						notifications.value.constructNotification(`Failed to replace contents of ${config.samba.confPath}`, errorStringHTML(error), 'error');
 						return;
 					}
 				} catch (error) {
-					notifications.value.constructNotification("Failed to back up /etc/samba/smb.conf", "Original config is unmodified.\n" + errorStringHTML(error), 'error');
+					notifications.value.constructNotification(`Failed to back up ${config.samba.confPath}`, "Original config is unmodified.\n" + errorStringHTML(error), 'error');
 					return;
 				}
 			}
@@ -410,6 +413,8 @@ export default {
 			processOutputDownload(['net', 'conf', 'list'], filename, { superuser: 'try' });
 		}
 
+		watch(() => config.samba.confPath, () => checkConf(), { immediate: true });
+
 		const watchHandles = [];
 
 		watchHandles.push(cockpit.file('/etc/ctdb/nodes', { superuser: 'try' }).watch(() => getCtdbHosts(), { read: false }));
@@ -417,6 +422,7 @@ export default {
 		onBeforeUnmount(() => watchHandles.map(handle => handle?.remove?.()));
 
 		return {
+			config,
 			shares,
 			globalConfig,
 			users,
diff --git a/file-sharing/src/components/SambaShareEditor.vue b/file-sharing/src/components/SambaShareEditor.vue
index 437dc1b..6e3d0fb 100644
--- a/file-sharing/src/components/SambaShareEditor.vue
+++ b/file-sharing/src/components/SambaShareEditor.vue
@@ -305,7 +305,7 @@ If not, see <https://www.gnu.org/licenses/>.
 <script>
 import PillList from "./PillList.vue";
 import DropdownSelector from "./DropdownSelector.vue";
-import { splitAdvancedSettings, joinAdvancedSettings, strToBool } from "../functions";
+import { splitAdvancedSettings, joinAdvancedSettings, strToBool, splitQuotedDelim } from "../functions";
 import { ChevronDownIcon, ExclamationCircleIcon, ExclamationIcon } from "@heroicons/vue/solid";
 import { ref, reactive, watch, inject, onMounted } from "vue";
 import { useSpawn, errorStringHTML, canonicalPath, systemdUnitEscape } from "@45drives/cockpit-helpers";
@@ -443,9 +443,7 @@ export default {
 					advancedSettings: []
 				}
 			);
-			tmpShare["valid users"]
-				.match(/(?:[^\s"]+|"[^"]*")+/g)
-				?.map(entity => entity.replace(/^"/, '').replace(/"$/, ''))
+			splitQuotedDelim(tmpShare["valid users"], ', ')
 				.forEach((entity) => {
 					if (entity.at(0) === '@') {
 						const groupName = entity.substring(1);
diff --git a/file-sharing/src/functions.js b/file-sharing/src/functions.js
index 56a6e62..da13040 100644
--- a/file-sharing/src/functions.js
+++ b/file-sharing/src/functions.js
@@ -63,3 +63,43 @@ export function strToBool(str) {
 	];
 	return trueStrings.includes(String(str).toLowerCase());
 }
+
+/**
+ * Split string by specified delimiters, ignoring delimiters in quotes
+ * Strips quotes from tokens
+ * @param {string} str - The string to split
+ * @param {string} delims - String of delimiters
+ * @returns {string[]} Array of tokens
+ */
+export function splitQuotedDelim(str, delims) {
+	const state = {
+		singleQuoted: false,
+		doubleQuoted: false,
+		/**
+		 * @type {string[]}
+		 */
+		tokens: [],
+	}
+	return str.split('').reduce((state, char, i, arr) => {
+		if (char === '"' && !state.singleQuoted) {
+			state.doubleQuoted = !state.doubleQuoted;
+		}
+		else if (char === '\'' && !state.doubleQuoted)
+			state.singleQuoted = !state.singleQuoted;
+		else if (delims.includes(char)) {
+			// found delim
+			if (state.doubleQuoted || state.singleQuoted) {
+				// found quoted delim
+				state.tokens.push((state.tokens.pop() ?? "") + char);
+			} else if (!delims.includes(arr[i - 1])) {
+				// start new token if first delim in a row
+				state.tokens.push("");
+			}
+			// else skip char
+		} else {
+			// append char to token
+			state.tokens.push((state.tokens.pop() ?? "") + char);
+		}
+		return state;
+	}, state).tokens;
+}
diff --git a/manifest.json b/manifest.json
index 9c41722..dcd45d4 100644
--- a/manifest.json
+++ b/manifest.json
@@ -3,7 +3,7 @@
     "name": "cockpit-file-sharing",
     "title": "Cockpit File Sharing",
     "prerelease": false,
-    "version": "3.2.3",
+    "version": "3.2.4",
     "buildVersion": "1",
     "author": "Josh Boudreau <jboudreau@45drives.com>",
     "url": "https://github.com/45Drives/cockpit-file-sharing",
@@ -58,7 +58,7 @@
     ],
     "changelog": {
         "urgency": "medium",
-        "version": "3.2.3",
+        "version": "3.2.4",
         "buildVersion": "1",
         "ignore": [],
         "date": null,
diff --git a/packaging/el8/main.spec b/packaging/el8/main.spec
index 2a205db..bb7e443 100644
--- a/packaging/el8/main.spec
+++ b/packaging/el8/main.spec
@@ -27,6 +27,9 @@ make DESTDIR=%{buildroot} install
 /usr/share/cockpit/file-sharing/*
 
 %changelog
+* Thu Aug 18 2022 Joshua Boudreau <jboudreau@45drives.com> 3.2.4-1
+- Add settings menu for configuring smb.conf and exports file locations
+- Fix parsing "valid users" field for Samba shares
 * Thu Aug 04 2022 Joshua Boudreau <jboudreau@45drives.com> 3.2.3-1
 - Fixed issue saving NFS exports file when /etc/exports.d DNE
 * Wed Jul 27 2022 Joshua Boudreau <jboudreau@45drives.com> 3.2.2-1
diff --git a/packaging/focal/changelog b/packaging/focal/changelog
index 3efcda0..0a9c46c 100644
--- a/packaging/focal/changelog
+++ b/packaging/focal/changelog
@@ -1,3 +1,10 @@
+cockpit-file-sharing (3.2.4-1focal) focal; urgency=medium
+
+  * Add settings menu for configuring smb.conf and exports file locations
+  * Fix parsing "valid users" field for Samba shares
+
+ -- Joshua Boudreau <jboudreau@45drives.com>  Thu, 18 Aug 2022 15:05:29 -0300
+
 cockpit-file-sharing (3.2.3-1focal) focal; urgency=medium
 
   * Fixed issue saving NFS exports file when /etc/exports.d DNE