[CAMEL-15764] Twilo camel case endpoint URI options leads to reflective property binding

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.6.0
• Fix Version/s: 3.7.0
• Component/s: camel-twilio
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: James Netherton
• Created: 27/Oct/20 11:08
• Updated: 05/Dec/20 14:10
• Resolved: 05/Dec/20 14:10

Description

I noticed some inconsistency when adding native support for Twilio in Camel Quarkus.

Consider the following URI:

twilio://incoming-phone-number/create?phoneNumber=RAW([phone])

Turns out that this leads to the phoneNumber property binding being done reflectively instead of via the configurer.

This problem is that ignoreCase is always false here:

https://github.com/apache/camel/blob/master/components/camel-twilio/src/generated/java/org/apache/camel/component/twilio/IncomingPhoneNumberEndpointConfigurationConfigurer.java#L33

Hence the case logic attempts to match on the exact parameter name, and there is no case block for the camel cased 'phoneNumber'.

The obvious workaround is to name the URI param 'phonenumber' or 'PhoneNumber' but the camel-twilio docs all refer to camel cased parameter names.

Issue Links

• is related to: CAMEL-15737 AS2 component URI parameter resolution requires reflection