Access is denied in Win32

[Zhao-Michael]

2026/01/06 16:20:57 generator.go:121: recv_generator(f=&{Name:top.key Length:1678 ModTime:2025-10-24 22:14:26 +0800 CST Mode:33188 Uid:0 Gid:0 LinkTarget: Rdev:0})
2026/01/06 16:20:57 generator.go:240: skipping cert\top.key
2026/01/06 16:20:57 generator.go:29: generateFiles phase=1
2026/01/06 16:20:57 generator.go:38: generateFiles phase=2
2026/01/06 16:20:57 generator.go:45: generateFiles finished
2026/01/06 16:20:57 receiver.go:35: receiving file idx=1: &{Name:pfx-password.txt Length:9 ModTime:2025-10-06 14:31:57 +0800 CST Mode:33188 Uid:0 Gid:0 LinkTarget: Rdev:0}
2026/01/06 16:20:57 receiver.go:103: creating cert\pfx-password.txt
2026/01/06 16:20:57 receiver.go:155: checksum 48f5bfd09404f68725f5cecf79c11e8b matches!
2026/01/06 16:20:57 rename cert\temp-rsync-230283779 cert\pfx-password.txt: Access is denied.

Linux Host (Source): ./gokr-rsync --daemon --gokr.listen=0.0.0.0:8730 --gokr.modulemap=home=/home/

Win10 Host (Dest): ./gokr-rsync.exe -av --port 8730 rsync://server_ip/home/cert/ ./cert

1. At the first sync, everything works fine
2. Then I add some content into text files at Win10
3. run sync cmd at Win10 again,it failed : 'rename cert\temp-rsync-230283779 cert\pfx-password.txt: Access is denied.'

[Zhao-Michael]

Two Linux Host all works good.

[stapelberg]

Hm, I am not sure what the problem is. What happens when you try to manually rename a temporary file over cert\pfx-password.txt in cmd.exe?

(https://github.com/dchest/safefile/blob/master/rename_nonatomic.go does a multi-rename dance on Windows. Maybe worth a try.)

[Zhao-Michael]

I just test it, use the rename function can work : https://github.com/dchest/safefile/blob/855e8d98f1852d48dde521e0522408d1fe7e836a/rename_nonatomic.go#L14C6-L14C12

[stapelberg]

Would it be possible to create a unit test for this case that works on GitHub Actions? Otherwise, we risk the functionality breaking over time, even if we fix individual issues.

[Zhao-Michael]

I’m not sure if the GitHub Windows headless runner can reproduce the problem. Maybe a practical solution is to add a toggle configuration for this fix?

[stapelberg]

I’m not sure if the GitHub Windows headless runner can reproduce the problem.

Why not?

Maybe a practical solution is to add a toggle configuration for this fix?

No, we will not add Windows-specific configuration in this project. It’s nice if the project works (somewhat) on Windows, but not a focus. I’m 100% on Linux.