Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Ruby - URL encoding

I have a URL string where there are line breaks and carriage returns. E.g.

http://xyz.com/hello?name=john&msg=hello\nJohn\n\rgoodmorning&note=last\night I went to \roger

Where my actual msg string is:

hello
john
goodmorning

and note string is

last\night I went to \roger

In order to send it correctly I have to urlencode this

 http://xyz.com/hello?name%3Djohn%26msg%3Dhello%5CnJohn%5Cn%5Crgoodmorning%26note%3Dlast%5Cnight%20I%20went%20to%20%5Croger

But this encode mess up \n and \r. While I expect \n should get converted to %0A and \r to %0D

I am writing code is ruby. And I tried to take help of Addressable::URI but no help yet. Other way could be replacing \n and \r manually to %0A and %0D respectively. But that replacement can replace valid character such as last\night to last%0Aight that I don't want. Can anyone suggest a better solution? Thanks.

like image 693
JVK Avatar asked Dec 04 '22 13:12

JVK


2 Answers

What about CGI::escape

You need to only encode the parameters though.

url = "http://xyz.com/hello?"
params = "name=john&msg=hello\nJohn\n\rgoodmorning&note=last\night I went to \roger"

puts "#{url}#{CGI::escape(params)}"
# => "http://xyz.com/hello?name%3Djohn%26msg%3Dhello%0AJohn%0A%0Dgoodmorning%26note%3Dlast%0Aight+I+went+to+%0Doger"
like image 124
oldergod Avatar answered Dec 28 '22 10:12

oldergod


This is how I'd do it using Addressable::URI:

require 'addressable/uri'

url = 'http://xyz.com/hello'

msg = 'hello
john
goodmorning'

note = "last\night I went to \roger"

uri = Addressable::URI.parse(url)
uri.query_values = {
  'msg'  => msg,
  'note' => note
}

puts uri.to_s

Which returns:

http://xyz.com/hello?msg=hello%0Ajohn%0Agoodmorning&note=last%0Aight%20I%20went%20to%20%0Doger

The \r in \roger and \n in \night are converted because I used a double-quote delimited string, instead of a single-quote delimited string, which would have preserved \r and \n as literals.

like image 23
the Tin Man Avatar answered Dec 28 '22 09:12

the Tin Man