8a776ad0b6
- dead time = 30: disconnect idle clients to free resources - large readwrite = yes: improved large file transfer performance - max xmit = 65535: maximum packet size for better throughput - write cache size = 1048576: 1MB write cache for write performance 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
56 lines
2.3 KiB
Markdown
56 lines
2.3 KiB
Markdown
# Changelog
|
|
|
|
All notable changes to this project will be documented in this file.
|
|
|
|
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
|
|
|
|
## [Unreleased]
|
|
|
|
### Added
|
|
|
|
- **Performance optimizations** (Dockerfile)
|
|
- `dead time = 30` — disconnect idle clients after 30 seconds to free resources
|
|
- `large readwrite = yes` — improved performance for large file transfers
|
|
- `max xmit = 65535` — maximum packet size for better throughput
|
|
- `write cache size = 1048576` — 1MB write cache for improved write performance
|
|
|
|
### Fixed
|
|
|
|
- **Fixed chown syntax error in perms function** (samba.sh:122)
|
|
- Changed `smbuser.` to `smbuser:smb` for correct group assignment
|
|
|
|
- **Fixed paths with spaces handling in perms function** (samba.sh:121)
|
|
- Replaced `for` loop with `while IFS= read -r` to correctly handle paths containing spaces
|
|
|
|
- **Fixed unquoted variables in import function** (samba.sh:112-113)
|
|
- Added quotes around `$file` variable to prevent word splitting issues
|
|
|
|
- **Removed /etc from VOLUME declaration** (Dockerfile:83)
|
|
- `/etc` is too broad and can cause unexpected behavior with system configurations
|
|
|
|
- **Share-specific parameters not overriding global settings** ([#issue](https://github.com/upagge/samba/issues))
|
|
|
|
**Problem**: When creating public shares with guest write access, the `-G` parameters for individual shares did not override the global `force user` and `force group` settings from the base `smb.conf`.
|
|
|
|
Example that didn't work:
|
|
```bash
|
|
-s "public;/cloud/share;yes;no;yes"
|
|
-G "public;force user = nobody"
|
|
-G "public;force group = nogroup"
|
|
```
|
|
|
|
`testparm -s` showed that the share used global `force user = smbuser` and `force group = smb` instead of the specified values.
|
|
|
|
**Root cause**:
|
|
1. When using environment variables, `GENERIC` was processed **before** `SHARE`, so share sections didn't exist when `-G` options tried to modify them.
|
|
2. The regex `\s` in sed was not POSIX-compatible for Alpine/busybox.
|
|
|
|
**Solution**:
|
|
- Reordered environment variable processing: `SHARE` is now processed before `GENERIC`
|
|
- Replaced `\s` with POSIX-compatible `[[:space:]]` in regex patterns
|
|
- Added `^` anchor to sed append command for precise matching
|
|
|
|
### Changed
|
|
|
|
- Environment variable processing order: `GLOBAL` -> `SHARE` -> `GENERIC` (was: `GENERIC` -> `GLOBAL` -> `SHARE`)
|