[
https://issues.apache.org/jira/browse/SSHD-929?focusedWorklogId=265796&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-265796
]
ASF GitHub Bot logged work on SSHD-929:
---------------------------------------
Author: ASF GitHub Bot
Created on: 24/Jun/19 16:23
Start Date: 24/Jun/19 16:23
Worklog Time Spent: 10m
Work Description: lgoldstein commented on pull request #104: [SSHD-929]
Create separate listener for remove directory.
URL: https://github.com/apache/mina-sshd/pull/104#discussion_r296805009
##########
File path:
sshd-sftp/src/main/java/org/apache/sshd/server/subsystem/sftp/SftpEventListener.java
##########
@@ -393,6 +393,30 @@ default void removed(ServerSession session, Path path,
Throwable thrown) throws
// ignored
}
+ /**
+ * Called <U>prior</U> to removing a directory
+ *
+ * @param session The {@link ServerSession} through which the request was
handled
+ * @param path The {@link Path} about to be removed
+ * @throws IOException If failed to handle the call
+ * @see #removed(ServerSession, Path, Throwable)
+ */
+ default void removingDirectory(ServerSession session, Path path) throws
IOException {
Review comment:
In order to really make the distinction, the original `removing/remove`
should also be renamed to `removingFile/removedFile`
----------------------------------------------------------------
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: 265796)
Time Spent: 20m (was: 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
> Priority: Minor
> Time Spent: 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]
