Skip to content

Handle FileExistsException in simpleRename without disconnecting slave#500

Open
tsautier wants to merge 1 commit into
drftpd-ng:masterfrom
tsautier:fix/issue-195
Open

Handle FileExistsException in simpleRename without disconnecting slave#500
tsautier wants to merge 1 commit into
drftpd-ng:masterfrom
tsautier:fix/issue-195

Conversation

@tsautier

@tsautier tsautier commented Jan 14, 2026

Copy link
Copy Markdown

Problem

Slave crashes when attempting to move a file that already exists at the destination (e.g., during affil-race + site pre). The FileExistsException from the IO operation was propagating uncaught.

Solution

Add specific handling for FileExistsException in the slave's async command handler to prevent crashes.

Changes Made

  • Modified exception handling to catch FileExistsException
  • Return appropriate error response instead of crashing the slave
  • IO errors are now gracefully handled without disconnecting

Fixes: #195

@tsautier

Copy link
Copy Markdown
Author

@mvangoor @hrxcodes for review

@tsautier tsautier changed the title Fix/issue 195 Handle FileExistsException in simpleRename without disconnecting slave Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Slave is killed on IO errors - File already exists

1 participant