From 6fda6ebad04005b8423a50e0bf94384ca2350aab Mon Sep 17 00:00:00 2001 From: "Torsten Schulz (local)" Date: Sat, 20 Dec 2025 15:05:49 +0100 Subject: [PATCH] Enhance security and error handling in various components by refining error catch blocks to ignore specific errors, improving code clarity and consistency across the application. --- .gitea/workflows/code-analysis.yml | 15 +++++++ components/Footer.vue | 2 +- components/MannschaftenUebersicht.vue | 4 +- components/Spielplan.vue | 2 +- middleware/auth.global.js | 2 +- middleware/auth.js | 4 +- pages/cms/benutzer.vue | 8 ++-- pages/cms/einstellungen.vue | 8 ++-- pages/cms/geschichte.vue | 4 +- pages/cms/spielplaene.vue | 6 +-- pages/cms/termine.vue | 4 +- pages/cms/tt-regeln.vue | 8 ++-- pages/cms/ueber-uns.vue | 4 +- pages/mannschaften/[slug].vue | 2 +- pages/mannschaften/spielplaene.vue | 8 ++-- pages/mitgliederbereich/mitglieder.vue | 4 +- pages/mitgliederbereich/profil.vue | 2 +- pages/newsletter/subscribe.vue | 2 +- pages/spielplan.vue | 2 +- pages/tt-regeln.vue | 2 +- pages/verein/geschichte.vue | 2 +- pages/verein/tt-regeln.vue | 2 +- pages/verein/ueber-uns.vue | 2 +- server/api/cms/satzung-upload.post.js | 2 +- server/api/config.get.js | 2 +- .../members/toggle-mannschaftsspieler.post.js | 2 +- server/api/membership/generate-pdf.post.js | 41 +++++++++---------- server/api/membership/update-status.put.js | 2 +- .../groups/[id]/posts/create.post.js | 3 +- .../newsletter/groups/[id]/posts/list.get.js | 5 +-- .../api/newsletter/groups/public-list.get.js | 2 +- server/api/profile.put.js | 2 +- server/api/spielplan.get.js | 2 +- .../api/spielplan/download/[filename].get.js | 2 +- server/api/spielplan/pdf.get.js | 8 ++-- server/utils/auth.js | 16 ++++---- server/utils/members.js | 6 +-- server/utils/newsletter.js | 4 +- server/utils/pdf-field-mapper.js | 2 +- server/utils/pdf-form-filler.js | 19 ++++----- server/utils/pdf-generator-service.js | 7 ++-- stores/auth.js | 4 +- 42 files changed, 117 insertions(+), 113 deletions(-) diff --git a/.gitea/workflows/code-analysis.yml b/.gitea/workflows/code-analysis.yml index 53121b3..f8c6d30 100644 --- a/.gitea/workflows/code-analysis.yml +++ b/.gitea/workflows/code-analysis.yml @@ -31,3 +31,18 @@ jobs: - name: Semgrep (SAST) run: semgrep --config p/default --error . + + - name: npm audit (high+) + run: npm audit --audit-level=high || true + + - name: OSV-Scanner (SCA) + run: | + curl -L -o osv-scanner https://github.com/google/osv-scanner/releases/latest/download/osv-scanner_linux_amd64 + chmod +x osv-scanner + ./osv-scanner --lockfile package-lock.json + + - name: gitleaks (Secrets Scanning) + run: | + curl -sSL https://github.com/gitleaks/gitleaks/releases/latest/download/gitleaks_linux_x64.tar.gz \ + | tar -xz gitleaks + ./gitleaks detect --source . --no-git --redact --exit-code 1 diff --git a/components/Footer.vue b/components/Footer.vue index 6e6abca..fa1e731 100644 --- a/components/Footer.vue +++ b/components/Footer.vue @@ -100,7 +100,7 @@ const isMemberMenuOpen = ref(false) // Reactive auth state from store const isLoggedIn = computed(() => authStore.isLoggedIn) -const isAdmin = computed(() => authStore.isAdmin) +// const isAdmin = computed(() => authStore.isAdmin) const toggleMemberMenu = () => { isMemberMenuOpen.value = !isMemberMenuOpen.value diff --git a/components/MannschaftenUebersicht.vue b/components/MannschaftenUebersicht.vue index aaf544e..45f3ea9 100644 --- a/components/MannschaftenUebersicht.vue +++ b/components/MannschaftenUebersicht.vue @@ -103,7 +103,7 @@