[
https://issues.apache.org/jira/browse/SSHD-929?focusedWorklogId=267210&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-267210
]
ASF GitHub Bot logged work on SSHD-929:
---------------------------------------
Author: ASF GitHub Bot
Created on: 26/Jun/19 01:48
Start Date: 26/Jun/19 01:48
Worklog Time Spent: 10m
Work Description: The-Yoda commented on pull request #104: [SSHD-929]
Create separate listener for remove directory.
URL: https://github.com/apache/mina-sshd/pull/104#discussion_r297456603
##########
File path:
sshd-sftp/src/main/java/org/apache/sshd/server/subsystem/sftp/AbstractSftpSubsystemHelper.java
##########
@@ -1631,8 +1650,8 @@ protected void doRemove(int id, String path,
LinkOption... options) throws IOExc
log.debug("signalRemovalPreConditionFailure(id={})[{}] signal {}
for {}: {}",
id, pathValue, thrown.getClass().getSimpleName(), path,
thrown.getMessage());
}
- listener.removing(session, path);
- listener.removed(session, path, thrown);
+ listener.removingFile(session, path);
+ listener.removedFile(session, path, thrown);
Review comment:
Thats right. It is being called for both directory and file. Fixed the same.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 267210)
Time Spent: 1h 20m (was: 1h 10m)
> Separate listener for deleting directory in SFTP
> ------------------------------------------------
>
> Key: SSHD-929
> URL: https://issues.apache.org/jira/browse/SSHD-929
> Project: MINA SSHD
> Issue Type: Improvement
> Affects Versions: 2.3.0
> Reporter: Subramaniajeeva
> Assignee: Goldstein Lyor
> Priority: Minor
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
> Right now listener for deletion of file and directory is common. This makes
> it difficult to find whether directory / file has been removed.
> The path passed to `removed` function in SftpEventListener will not tell if
> the deleted path was a directory or file. `Files.isDirectory(path)` will
> return false, as the derectory doesn't exist.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
