URI parser cannot parse nil path for CarrierWave::Storage::Fog::File#public_url

[tpdogan]

Hello there,

After upgrading to the latest version of carrierwave, CarrierWave::Storage::Fog::File#public_url started to raise undefined method 'gsub' for nil:NilClass for URI parser. It seems that this is due to this commit, which changed usage of path from path.to_s to path for #encode_path method. Thus, the use case of path being nil is not considered.

Following line

carrierwave/lib/carrierwave/utilities/uri.rb

Line 14 in 8b33235

URI::DEFAULT_PARSER.escape(path, PATH_UNSAFE)

should be

URI::DEFAULT_PARSER.escape(path.to_s, PATH_UNSAFE)

[abepark01]

@tpdogan thanks! I changed the gem locally, but the fix worked for me. CC: @mshibuya

[abepark01]

@tpdogan @mshibuya I created a small PR to fix this regression per @tpdogan's insights.

[mshibuya]

I'd like to understand one thing.
When path becomes nil? How do you configure CarrierWave to behave so?