close
Skip to content

fix: remove "nextcloud" from debug log filenames#8899

Merged
nilsding merged 1 commit into
masterfrom
bugfix/DebugLogFiles
Oct 20, 2025
Merged

fix: remove "nextcloud" from debug log filenames#8899
nilsding merged 1 commit into
masterfrom
bugfix/DebugLogFiles

Conversation

@Rello
Copy link
Copy Markdown
Collaborator

@Rello Rello commented Oct 20, 2025

The first two text files have a hardcoded "nextcloud" also for branded client.
changing to neutral

_client_buildinfo
_client_pramaters

do we need them at all?

Bildschirmfoto 2025-10-20 um 10 25 41

Signed-off-by: Rello <Rello@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-8899.zip

Digest: sha256:b9a6090b50a819b7e4f90fd2ee6394099b1119843db541eaff537e822ab7f1c7

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
BERJAYA 0.0% Coverage on New Code (required ≥ 80%)
BERJAYA E Maintainability Rating on New Code (required ≥ A)
BERJAYA 23 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

BERJAYA Catch issues before they fail your Quality Gate with our IDE extension BERJAYA SonarQube for IDE

@nilsding nilsding merged commit 92c87d4 into master Oct 20, 2025
19 of 22 checks passed
@nilsding nilsding deleted the bugfix/DebugLogFiles branch October 20, 2025 07:01
@nilsding
Copy link
Copy Markdown
Member

/backport to stable-4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants