Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: 7zip headers warning with nextcloud 31 generated zip files #51180

Open
5 of 8 tasks
ivanjx opened this issue Mar 2, 2025 · 4 comments
Open
5 of 8 tasks

[Bug]: 7zip headers warning with nextcloud 31 generated zip files #51180

ivanjx opened this issue Mar 2, 2025 · 4 comments
Labels
1. to develop Accepted and waiting to be taken care of 31-feedback bug feature: files

Comments

@ivanjx
Copy link

ivanjx commented Mar 2, 2025

⚠️ This issue respects the following points: ⚠️

Bug description

i am trying to download multiple files from a public shared folder, which in turn creates a zip file and that file gets downloaded. when trying to extract it with 7zip (or nanazip from ms store) i get header errors warning. this does not happen with nextcloud 30 and prior.

Steps to reproduce

  1. upload multiple files into a folder.
  2. share the folder.
  3. open the shared folder link and select all files.
  4. click download.
  5. try to extract the downloaded file with 7zip or nanazip.

Expected behavior

no warnings

Nextcloud Server version

31

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.4

Web server

Nginx

Database engine version

MariaDB

Is this bug present after an update or on a fresh install?

Upgraded to a MAJOR version (ex. 31 to 32)

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

{
    "system": {
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_proxies": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
        ],
        "dbtype": "mysql",
        "version": "31.0.0.18",
        "overwrite.cli.url": "http:\/\/files.ivanjkara.net",
        "dbname": "***REMOVED SENSITIVE VALUE***",
        "dbhost": "***REMOVED SENSITIVE VALUE***",
        "dbport": "",
        "dbtableprefix": "oc_",
        "mysql.utf8mb4": true,
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "installed": true,
        "memcache.local": "\\OC\\Memcache\\APCu",
        "memcache.distributed": "\\OC\\Memcache\\Redis",
        "memcache.locking": "\\OC\\Memcache\\Redis",
        "filelocking.enabled": true,
        "redis": {
            "host": "***REMOVED SENSITIVE VALUE***",
            "port": 6379
        },
        "upgrade.disable-web": true,
        "maintenance_window_start": 4,
        "log.condition": {
            "apps": [
                "admin_audit"
            ]
        },
        "log_type": "file",
        "logfile": "\/data\/nextcloud.log",
        "log_type_audit": "file",
        "logfile_audit": "\/data\/nextcloud.log",
        "simpleSignUpLink.shown": false,
        "maintenance": false,
        "loglevel": 2,
        "trashbin_retention_obligation": "30, 35",
        "files.chunked_upload.max_size": 20971520,
        "app_install_overwrite": [
            "checksum"
        ]
    }
}

List of activated Apps

Enabled:
  - activity: 4.0.0
  - admin_audit: 1.21.0
  - app_api: 5.0.2
  - bookmarks: 15.1.0
  - bruteforcesettings: 4.0.0
  - checksum: 1.2.5
  - circles: 31.0.0-dev.0
  - cloud_federation_api: 1.14.0
  - comments: 1.21.0
  - contactsinteraction: 1.12.0
  - dav: 1.33.0
  - deck: 1.15.0
  - federatedfilesharing: 1.21.0
  - federation: 1.21.0
  - files: 2.3.1
  - files_downloadlimit: 4.0.0
  - files_external: 1.23.0
  - files_pdfviewer: 4.0.0
  - files_reminders: 1.4.0
  - files_sharing: 1.23.1
  - files_trashbin: 1.21.0
  - files_versions: 1.24.0
  - firstrunwizard: 4.0.0
  - groupfolders: 19.0.3
  - impersonate: 2.0.0
  - logreader: 4.0.0
  - lookup_server_connector: 1.19.0
  - nextcloud_announcements: 3.0.0
  - notes: 4.11.0
  - notifications: 4.0.0
  - oauth2: 1.19.1
  - password_policy: 3.0.0
  - privacy: 3.0.0
  - profile: 1.0.0
  - provisioning_api: 1.21.0
  - recommendations: 4.0.0
  - related_resources: 2.0.0
  - richdocuments: 8.6.2
  - serverinfo: 3.0.0
  - settings: 1.14.0
  - sharebymail: 1.21.0
  - sociallogin: 5.9.2
  - spreed: 21.0.0
  - support: 3.0.0
  - survey_client: 3.0.0
  - systemtags: 1.21.1
  - text: 5.0.0
  - theming: 2.6.1
  - twofactor_backupcodes: 1.20.0
  - twofactor_totp: 13.0.0-dev.0
  - updatenotification: 1.21.0
  - user_status: 1.11.0
  - viewer: 4.0.0
  - webhook_listeners: 1.2.0
  - workflowengine: 2.13.0
Disabled:
  - dashboard: 7.11.0 (installed 7.9.0)
  - encryption: 2.19.0
  - photos: 4.0.0-dev.1 (installed 2.5.0)
  - suspicious_login: 9.0.1
  - twofactor_nextcloud_notification: 5.0.0
  - user_ldap: 1.22.0
  - weather_status: 1.11.0 (installed 1.9.0)

Nextcloud Signing status

no errors

Nextcloud Logs

irrelevant

Additional info

Image

@ivanjx ivanjx added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Mar 2, 2025
@susnux
Copy link
Contributor

susnux commented Mar 2, 2025

Just a guess (for internal purpose): could be related to zip64 compression.

@solracsf
Copy link
Member

solracsf commented Mar 2, 2025

Could this be related to #49035 ?
Maybe you @ivanjx can revert that small change and give it a try?

@ivanjx
Copy link
Author

ivanjx commented Mar 4, 2025

i am using the lsio's docker image. how can i do that?

@susnux
Copy link
Contributor

susnux commented Mar 5, 2025

Could this be related to #49035 ?

I do not think so or if than something other is wrong there. As that only enabled zip64 but here some headers are wrong - at least for 7zip which is stricter on header validity.
Should still work as this is just a warning.

@susnux susnux added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap needs info labels Mar 5, 2025
@susnux susnux changed the title [Bug]: 7zip headers error with nextcloud 31 generated zip files [Bug]: 7zip headers warning with nextcloud 31 generated zip files Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of 31-feedback bug feature: files
Projects
None yet
Development

No branches or pull requests

4 participants