From 171f24505ee08aaea80290df52aabf2c800afa3a Mon Sep 17 00:00:00 2001
From: "Grigorii K. Shartsev" <me@shgk.me>
Date: Thu, 25 Jul 2024 10:53:30 +0200
Subject: [PATCH] fix(AdminSettings): hide secrets in password fields

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
---
 src/components/AdminSettings/RecordingServers.vue | 7 ++++---
 src/components/AdminSettings/SIPBridge.vue        | 8 +++++---
 src/components/AdminSettings/SignalingServers.vue | 7 ++++---
 src/components/AdminSettings/TurnServer.vue       | 5 ++++-
 4 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/src/components/AdminSettings/RecordingServers.vue b/src/components/AdminSettings/RecordingServers.vue
index 543350d90a1..e883c595276 100644
--- a/src/components/AdminSettings/RecordingServers.vue
+++ b/src/components/AdminSettings/RecordingServers.vue
@@ -57,9 +57,10 @@
 			{{ t('spreed', 'Add a new recording backend server') }}
 		</NcButton>
 
-		<NcTextField class="form__textfield additional-top-margin"
+		<NcPasswordField class="form__textfield additional-top-margin"
 			:value="secret"
 			name="recording_secret"
+			autocomplete="new-password"
 			:disabled="loading"
 			:placeholder="t('spreed', 'Shared secret')"
 			:label="t('spreed', 'Shared secret')"
@@ -101,7 +102,7 @@ import { loadState } from '@nextcloud/initial-state'
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/dist/Components/NcCheckboxRadioSwitch.js'
 import NcNoteCard from '@nextcloud/vue/dist/Components/NcNoteCard.js'
-import NcTextField from '@nextcloud/vue/dist/Components/NcTextField.js'
+import NcPasswordField from '@nextcloud/vue/dist/Components/NcPasswordField.js'
 
 import RecordingServer from '../../components/AdminSettings/RecordingServer.vue'
 import TransitionWrapper from '../TransitionWrapper.vue'
@@ -122,7 +123,7 @@ export default {
 		NcButton,
 		NcCheckboxRadioSwitch,
 		NcNoteCard,
-		NcTextField,
+		NcPasswordField,
 		Plus,
 		RecordingServer,
 		TransitionWrapper,
diff --git a/src/components/AdminSettings/SIPBridge.vue b/src/components/AdminSettings/SIPBridge.vue
index f2d910df740..ee950805d3b 100644
--- a/src/components/AdminSettings/SIPBridge.vue
+++ b/src/components/AdminSettings/SIPBridge.vue
@@ -62,9 +62,11 @@
 			<label for="sip-shared-secret" class="form__label additional-top-margin">
 				{{ t('spreed', 'Shared secret') }}
 			</label>
-			<NcTextField id="sip-shared-secret"
+			<NcPasswordField id="sip-shared-secret"
 				:value.sync="sharedSecret"
 				class="form"
+				name="sip-shared-secret"
+				autocomplete="new-password"
 				:disabled="loading"
 				:placeholder="t('spreed', 'Shared secret')"
 				label-outside />
@@ -104,9 +106,9 @@ import { generateOcsUrl } from '@nextcloud/router'
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/dist/Components/NcCheckboxRadioSwitch.js'
 import NcNoteCard from '@nextcloud/vue/dist/Components/NcNoteCard.js'
+import NcPasswordField from '@nextcloud/vue/dist/Components/NcPasswordField.js'
 import NcSelect from '@nextcloud/vue/dist/Components/NcSelect.js'
 import NcTextArea from '@nextcloud/vue/dist/Components/NcTextArea.js'
-import NcTextField from '@nextcloud/vue/dist/Components/NcTextField.js'
 
 import { setSIPSettings } from '../../services/settingsService.js'
 import { getWelcomeMessage } from '../../services/signalingService.js'
@@ -120,7 +122,7 @@ export default {
 		NcNoteCard,
 		NcSelect,
 		NcTextArea,
-		NcTextField,
+		NcPasswordField,
 	},
 
 	data() {
diff --git a/src/components/AdminSettings/SignalingServers.vue b/src/components/AdminSettings/SignalingServers.vue
index c416ba5d8ce..151990e0550 100644
--- a/src/components/AdminSettings/SignalingServers.vue
+++ b/src/components/AdminSettings/SignalingServers.vue
@@ -71,9 +71,10 @@
 			</NcCheckboxRadioSwitch>
 		</template>
 
-		<NcTextField class="form__textfield additional-top-margin"
+		<NcPasswordField class="form__textfield additional-top-margin"
 			:value="secret"
 			name="signaling_secret"
+			autocomplete="new-password"
 			:disabled="loading"
 			:placeholder="t('spreed', 'Shared secret')"
 			:label="t('spreed', 'Shared secret')"
@@ -93,7 +94,7 @@ import { loadState } from '@nextcloud/initial-state'
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/dist/Components/NcCheckboxRadioSwitch.js'
 import NcNoteCard from '@nextcloud/vue/dist/Components/NcNoteCard.js'
-import NcTextField from '@nextcloud/vue/dist/Components/NcTextField.js'
+import NcPasswordField from '@nextcloud/vue/dist/Components/NcPasswordField.js'
 
 import SignalingServer from '../../components/AdminSettings/SignalingServer.vue'
 import TransitionWrapper from '../TransitionWrapper.vue'
@@ -107,7 +108,7 @@ export default {
 		NcButton,
 		NcCheckboxRadioSwitch,
 		NcNoteCard,
-		NcTextField,
+		NcPasswordField,
 		Plus,
 		SignalingServer,
 		TransitionWrapper,
diff --git a/src/components/AdminSettings/TurnServer.vue b/src/components/AdminSettings/TurnServer.vue
index bb491ea81f7..ed489aede53 100644
--- a/src/components/AdminSettings/TurnServer.vue
+++ b/src/components/AdminSettings/TurnServer.vue
@@ -46,8 +46,9 @@
 			:label="t('spreed', 'TURN server URL')"
 			@update:value="updateServer" />
 
-		<NcTextField ref="turn_secret"
+		<NcPasswordField ref="turn_secret"
 			name="turn_secret"
+			autocomplete="new-password"
 			placeholder="secret"
 			class="turn-server__textfield"
 			:value="secret"
@@ -101,6 +102,7 @@ import Check from 'vue-material-design-icons/Check.vue'
 import Delete from 'vue-material-design-icons/Delete.vue'
 
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
+import NcPasswordField from '@nextcloud/vue/dist/Components/NcPasswordField.js'
 import NcSelect from '@nextcloud/vue/dist/Components/NcSelect.js'
 import NcTextField from '@nextcloud/vue/dist/Components/NcTextField.js'
 import Tooltip from '@nextcloud/vue/dist/Directives/Tooltip.js'
@@ -124,6 +126,7 @@ export default {
 		NcButton,
 		NcSelect,
 		NcTextField,
+		NcPasswordField,
 	},
 
 	props: {