Skip to content

Conversation

@dennysubke
Copy link
Contributor

Update Seafile to version 13.0.12

  • Added missing is_department_owner field to align community and pro database schemas
  • Fixed LDAP and LDAP (Imported) pages in system admin panel
  • Fixed issue where wiki module was shown even when disabled
  • Fixed XSS vulnerability in SVG file view for sharing links when using the Go file server
  • Added support for opening DOCXF files with OnlyOffice
  • Various UI fixes and consistency improvements

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

🎉   Linting finished with no errors or warnings   🎉

Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store.

Please review the linting results below and make any necessary changes to your submission.

Linting Results

Severity File Description
ℹ️ seafile/docker-compose.yml External port mapping "8921:80":
Port mappings may be unnecessary for the app to function correctly. Docker's internal DNS resolves container names to IP addresses within the same network. External access to the web interface is handled by the app_proxy container. Port mappings are only needed if external access is required to a port not proxied by the app_proxy, or if an app needs to expose multiple ports for its functionality (e.g., DHCP, DNS, P2P, etc.).
ℹ️ seafile/docker-compose.yml Potentially using unsafe user in service "db":
The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000.
ℹ️ seafile/docker-compose.yml Potentially using unsafe user in service "memcached":
The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000.
ℹ️ seafile/docker-compose.yml Potentially using unsafe user in service "seafile":
The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000.
ℹ️ seafile/docker-compose.yml Potentially using unsafe user in service "seadoc":
The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000.

Legend

Symbol Description
Error: This must be resolved before this PR can be merged.
⚠️ Warning: This is highly encouraged to be resolved, but is not strictly mandatory.
ℹ️ Info: This is just for your information.

@al-lac
Copy link
Collaborator

al-lac commented Nov 7, 2025

Hey @dennysubke, thanks for contributing this update! 🚀

I tried to install it but got errors in the logs. Does it start for you?

I think there is some more effort needed to upgrade from version 12 to 13: https://manual.seafile.com/13.0/upgrade/upgrade_notes_for_13.0.x/

@al-lac al-lac added the awaiting changes Waiting for contributor to make requested changes label Nov 7, 2025
@dennysubke
Copy link
Contributor Author

Hey @al-lac, thanks for testing and for the feedback! 🙏

It doesn’t start for me either. I ran into the same issue.
You’re right, the upgrade from Seafile 12 → 13 isn’t a simple image bump and needs additional migration steps.

I’ll take a closer look at the upgrade notes and check what’s required to make this work properly. 💪🏻

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

Labels

awaiting changes Waiting for contributor to make requested changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants