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

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

    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

          People

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

            Dates

              Created:
              Updated:
              Resolved: