luigidemasi commented on a change in pull request #4573:
URL: https://github.com/apache/camel/pull/4573#discussion_r518816293
##########
File path:
components/camel-file/src/main/java/org/apache/camel/component/file/GenericFileDefaultMoveExistingFileStrategy.java
##########
@@ -49,14 +49,24 @@ public boolean moveExistingFile(GenericFileEndpoint
endpoint, GenericFileOperati
dummy.getIn().setHeader(Exchange.FILE_PARENT, parent);
String to = endpoint.getMoveExisting().evaluate(dummy, String.class);
- // we must normalize it (to avoid having both \ and / in the name which
- // confuses java.io.File)
- to = FileUtil.normalizePath(to);
+
if (ObjectHelper.isEmpty(to)) {
throw new GenericFileOperationFailedException(
"moveExisting evaluated as empty String, cannot move
existing file: " + fileName);
}
+ if (to.length() > 1 && to.endsWith("/")) {
Review comment:
the problem is where to put the common logic, FileUtils doesn't seem a
good fit.
----------------------------------------------------------------
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:
us...@infra.apache.org
