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.

Sign up for GitHub

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

[ON HOLD] - Update syncthing to version v1.29.0 #2029

Closed
wants to merge 1 commit into from
Closed

[ON HOLD] - Update syncthing to version v1.29.0 #2029

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion syncthing/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
APP_PORT: 8384

server:
image: syncthing/syncthing:1.28.1@sha256:289b4ca86d77e4938d3e0af7d11f5c0a0fb786e469d5f697c25ab0f9e1f29f34
image: syncthing/syncthing:1.29.0@sha256:289b4ca86d77e4938d3e0af7d11f5c0a0fb786e469d5f697c25ab0f9e1f29f34
restart: on-failure
stop_grace_period: 1m
hostname: umbrel
Expand All @@ -16,5 +16,5 @@
- PGID=1000
volumes:
- ${APP_DATA_DIR}/data:/var/syncthing
ports:

Check notice on line 19 in syncthing/docker-compose.yml

View workflow job for this annotation

GitHub Actions / Lint apps

External port mapping "22000:22000/tcp" 

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.).

Check notice on line 19 in syncthing/docker-compose.yml

View workflow job for this annotation

GitHub Actions / Lint apps

External port mapping "22000:22000/udp" 

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.).
- 22000:22000/tcp
Expand Down
11 changes: 5 additions & 6 deletions syncthing/umbrel-app.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ manifestVersion: 1
id: syncthing
category: files
name: Syncthing
version: "1.28.1"
version: "1.29.0"
tagline: Peer-to-peer file synchronization between your devices
description: >-
Syncthing is a peer-to-peer continuous file synchronization
Expand All @@ -27,11 +27,10 @@ path: ""
defaultPassword: ""
releaseNotes: >-
This release includes bug fixes and improvements:
- Fixed issues with folder synchronization status display
- Improved dark theme support in the web interface
- Enhanced handling of symbolic links on Android
- Fixed various UI display issues on mobile devices
- Improved handling of directory junctions
- Fixed issues with socket file handling on Linux systems
- Improved GUI address handling for Unix sockets
- Enhanced folder synchronization behavior
- Clarified system log messages


Full release notes can be found at https://github.com/syncthing/syncthing/releases
Expand Down
Loading