You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by trrosa <gi...@git.apache.org> on 2018/03/25 23:17:31 UTC

[GitHub] camel pull request #2269: Fix DropboxAPIFacade.java

GitHub user trrosa opened a pull request:

    https://github.com/apache/camel/pull/2269

    Fix DropboxAPIFacade.java

    Specifically the bug was in class DropboxAPIFacade.java lines 114, 115, 142, 143. The exception thrown is about upload a file into a not file path or a folder into a not folder path, but the new version of dropbox java sdk don't provides the method entry.isFile() or entry.isFolder(), and the code as is now, is throwing erroneous exceptions, if the file exists remotely, a exception will be thrown, and in config, we have the force mode, i think this snipped of code should be erased or the mode=force param needs to be verified.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/trrosa/camel patch-1

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/2269.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2269
    
----
commit 09f69ed8f901f0167a7fcfce9e7715ef94462f77
Author: Tiago Rodrigues da Rosa <32...@...>
Date:   2018-03-25T23:17:05Z

    Fix DropboxAPIFacade.java
    
    Specifically the bug was in class DropboxAPIFacade.java lines 114, 115, 142, 143. The exception thrown is about upload a file into a not file path or a folder into a not folder path, but the new version of dropbox java sdk don't provides the method entry.isFile() or entry.isFolder(), and the code as is now, is throwing erroneous exceptions, if the file exists remotely, a exception will be thrown, and in config, we have the force mode, i think this snipped of code should be erased or the mode=force param needs to be verified.

----


---

[GitHub] camel pull request #2269: Fix DropboxAPIFacade.java

Posted by davsclaus <gi...@git.apache.org>.
Github user davsclaus closed the pull request at:

    https://github.com/apache/camel/pull/2269


---