chore(App): refactor requests

Signed-off-by: Dorra Jaouad <[email protected]>

src/App.vue

@@ -74,18 +74,21 @@
 </template>
 
 <script>
-import axios from '@nextcloud/axios'
 import { showError } from '@nextcloud/dialogs'
 import { loadState } from '@nextcloud/initial-state'
 import { t } from '@nextcloud/l10n'
-import { generateUrl } from '@nextcloud/router'
 import NcButton from '@nextcloud/vue/components/NcButton'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/components/NcCheckboxRadioSwitch'
 import NcInputField from '@nextcloud/vue/components/NcInputField'
 import NcNoteCard from '@nextcloud/vue/components/NcNoteCard'
 import NcSettingsSection from '@nextcloud/vue/components/NcSettingsSection'
 import PlusIcon from 'vue-material-design-icons/Plus.vue'
 import BruteForceItem from './components/BruteForceItem.vue'
+import {
+	addWhitelist,
+	deleteWhitelist,
+	getWhitelist,
+} from './services/bruteforceSettingsServices.js'
 
 export default {
 	name: 'App',
@@ -147,31 +150,24 @@ export default {
 		this.isBypassListed = loadState('bruteforcesettings', 'bypass-listed', false)
 		this.delay = loadState('bruteforcesettings', 'delay', 0)
 		this.isApplyAllowListToRateLimitEnabled = loadState('bruteforcesettings', 'apply_allowlist_to_ratelimit', false)
-
-		axios.get(generateUrl('apps/bruteforcesettings/ipwhitelist'))
-			.then((response) => {
-				this.items = response.data
-			})
+		this.loadWhitelist()
 	},
 
 	methods: {
 		t,
-		deleteWhitelist(id) {
-			axios.delete(generateUrl('apps/bruteforcesettings/ipwhitelist/{id}', { id }))
-				.then(() => {
-					this.items = this.items.filter((item) => item.id !== id)
-				})
+		async loadWhitelist() {
+			const response = await getWhitelist()
+			this.items = response.data
+		},
+
+		async deleteWhitelist(id) {
+			await deleteWhitelist(id)
+			this.items = this.items.filter((item) => item.id !== id)
 		},
 
 		async addWhitelist() {
 			try {
-				const response = await axios.post(
-					generateUrl('apps/bruteforcesettings/ipwhitelist'),
-					{
-						ip: this.newWhitelist.ip,
-						mask: this.newWhitelist.mask,
-					},
-				)
+				const response = await addWhitelist(this.newWhitelist.ip, this.newWhitelist.mask)
 
 				this.items.push(response.data)
 				this.newWhitelist.ip = ''

src/services/bruteforceSettingsServices.js

@@ -0,0 +1,39 @@
+/*
+ * SPDX-FileCopyrightText: 2025 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+import axios from '@nextcloud/axios'
+import { generateUrl } from '@nextcloud/router'
+
+/**
+ * Get the current IP whitelist
+ */
+export async function getWhitelist() {
+	return axios.get(generateUrl('apps/bruteforcesettings/ipwhitelist'))
+}
+
+/**
+ * Add a new IP to the whitelist
+ *
+ * @param {string} ip IP to add
+ * @param {number} mask Mask to add
+ */
+export async function addWhitelist(ip, mask) {
+	return axios.post(
+		generateUrl('apps/bruteforcesettings/ipwhitelist'),
+		{
+			ip,
+			mask,
+		},
+	)
+}
+
+/**
+ * Delete an IP from the whitelist
+ *
+ * @param {string} id ID of the IP to delete
+ */
+export async function deleteWhitelist(id) {
+	return axios.delete(generateUrl(`apps/bruteforcesettings/ipwhitelist/${id}`))
+}