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

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

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.20.0
    • Fix Version/s: 2.20.1, 2.21.0
    • Component/s: camel-dropbox
    • Labels:
    • Environment:

      all

    • Estimated Complexity:
      Novice
    • Flags:
      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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: