[CAMEL-11912] Camel Dropbox validator regex is too restrictive and fails for common paths

• 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: easyfix

Environment

all

• Estimated Complexity: Novice
• Flags: Important
• Assignee: Claus Ibsen
• Reporter: Thanos Agelatos
• Created: 16/Oct/17 08:43
• Updated: 20/Oct/17 10:44
• Resolved: 20/Oct/17 10:44

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.