InvalidURIError - how to clean up URIs before using .open()

rubyonrails-talk
#1

Hello, I got the following back from delayed_job:

    [Worker(XXXXXX pid:3720)] Class#XXXXXXX failed with
URI::InvalidURIError: bad URI(is not URI?):
https://s3.amazonaws.com/cline-local-dev/2/attachments/542/original/mac-os-x[1].jpeg?AWSAccessKeyId=xxxxxxxx&Expires=1295403309&Signature=xxxxxxx%3D
- 3 failed attempts

The way this URI comes from in my app is:

In my user_mailer I do:

      @comment.attachments.each do |a|
        attachments[a.attachment_file_name] =
open(a.authenticated_url()) {|f| f.read }
      end

Then in my attachments model:

      def authenticated_url(style = nil, expires_in = 90.minutes)
        AWS::S3::S3Object.url_for(attachment.path(style ||
attachment.default_style), attachment.bucket_name, :expires_in =>
expires_in, :use_ssl => attachment.s3_protocol == 'https')
      end

That being said, is there some type of URI.encode or parsing I can do
to prevent a valid URI (as I checked the URL works in my browser) for
erroring and killing delayed_job in rails 3?

Thank you!

#2

erroring...

and, sorry, that *doesn't* mean it's valid.

via: http://www.ietf.org/rfc/rfc1738.txt

<q>
Other characters are unsafe because
   gateways and other transport agents are known to sometimes modify
   such characters. These characters are "{", "}", "|", "\", "^", "~",
   "[", "]", and "`".

   All unsafe characters must always be encoded within a URL.
</q>

HTH,

#3

Thank you. What's the best way to Encode All Unsafe Characters?
And should that be done in the model? or in the mailer?