Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-11912

Camel Dropbox validator regex is too restrictive and fails for common paths

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.20.0
    • 2.20.1, 2.21.0
    • camel-dropbox
    • all

    • Novice
    • Important

    Description

      Most operations on the new Dropbox component (compliant to v2 API of Dropbox) has a dropbox path validator. It uses this regex (see below).

      /*?(\S+)/*?
      

      This regex fails in the following paths:

      • "/path with spaces/file" - part of the test suite
      • "/Web Projects/caesars entertainment - us/#ops/one/campaigns/archive/20170919/REN_BIG_PULL_WEEKEND_20170907.csv" - a real path that in the previous camel version (2.18.2) - v1 API was working.

      It seems that modifying the regex to this one below, (again regex form, not java) makes both paths above get a match.

      /*?(\S+)*.+/*?
      

      Please check (http://fiddle.re/f1c4fd) and let me know, patch can be provided if needed.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            davsclaus Claus Ibsen
            thagg1975 Thanos Agelatos
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment