Regression from v1 to v2 phone number lookups-- URI no longer encoded

[bf4]

Issue Summary

Same issue as https://github.com/twilio/twilio-ruby/issues/406

I see from v1 https://github.com/twilio/twilio-ruby/blob/6.0.2/lib/twilio-ruby/rest/lookups/v1/phone_number.rb#L53 to v2 the escaping was removed.

-                        @uri = "/PhoneNumbers/#{CGI.escape(@solution[:phone_number]).gsub("+", "%20")}"
+                        @uri = "/PhoneNumbers/#{@solution[:phone_number]}"

Proposal

use URI::DEFAULT_PARSER.escape just like Rails does

                        @uri = "/PhoneNumbers/#{URI::DEFAULT_PARSER.escape(@solution[:phone_number])}"

Steps to Reproduce

1. look up a phone number with a space in it

see https://github.com/twilio/twilio-ruby/issues/406

Code Snippet

see https://github.com/twilio/twilio-ruby/issues/406

Exception/Log

see https://github.com/twilio/twilio-ruby/issues/406

Technical details:

• twilio-ruby version: 6.0.1
• ruby version: 3.1

[AsabuHere]

Hi @bf4,

Thanks for the heads up!

This issue has been added to our internal backlog to be prioritised. Pull requests and +1s on the issue summary will help it move up the backlog.

With best regards,

Athira