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

Twilo camel case endpoint URI options leads to reflective property binding

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

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 3.6.0
    • 3.7.0
    • camel-twilio
    • None
    • Unknown

    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(+15005550006)

      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.

      Attachments

        Issue Links

        Activity

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

          People

            davsclaus Claus Ibsen
            jamesnetherton James Netherton
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment