René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Bad URI when a file name has brackets","articleBody":"When I try to add a watermark to a file with `[` or `]` in its name, I get `#\u003cURI::InvalidURIError: bad URI (is not URI?): \"https://v2.convertapi.com/d/p2bempn0sjua63w36zyxjr2d2weq7o34/[dev]+sample.pdf\"\u003e`\n\n```ruby\n\nConvertApi.convert(\n  \"watermark-overlay\",\n  {\n    File: \"./[dev] sample.pdf\",\n    OverlayFile: ConvertApi::UploadIO.new(File.open(\"some-other.pdf\")),\n    Scale: \"75\",\n    Opacity: \"30\",\n    Style: \"stamp\"\n  },\n  from_format: \"pdf\"\n).file.save(\"./sample.watermarked.pdf\")\n#\u003cURI::InvalidURIError: bad URI (is not URI?): \"https://v2.convertapi.com/d/p2bempn0sjua63w36zyxjr2d2weq7o34/[dev]+sample.pdf\"\u003e\n```\n\nIf I try to do it without saving, it will be OK, but the URI provided is not a valid URI nor URL:\n\n```ruby\nConvertApi.convert(\n  \"watermark-overlay\",\n  {\n    File: \"./[dev] sample.pdf\",\n    OverlayFile: ConvertApi::UploadIO.new(File.open(SaasAppSettings.instance.watermark_file_path)),\n    Scale: \"75\",\n    Opacity: \"30\",\n    Style: \"stamp\"\n  },\n  from_format: \"pdf\"\n).file\n\n#\u003cConvertApi::ResultFile:0x0000ffff94106a28\n# @info=\n#  {\"FileName\"=\u003e\"[dev] sample.pdf\",\n#   \"FileExt\"=\u003e\"pdf\",\n#   \"FileSize\"=\u003e121463,\n#   \"FileId\"=\u003e\"y0xnqdx161tcmbghnf0x3yjniih0ckek\",\n#   \"Url\"=\u003e\"https://v2.convertapi.com/d/y0xnqdx161tcmbghnf0x3yjniih0ckek/[dev]%20sample.pdf\"}\u003e \u003c= invalid!\n# but it should be 'https://v2.convertapi.com/d/f82j2d0chmxv4xqqa45s3kql613al2aj/%5Bdev%5D+sample.pdf'\n```\n\nSo I think it should be fixed on the server. The workaround I found is to patch the `ConvertApi::ResultFile`:\n```ruby\nrequire \"addressable/uri\"\n\nmodule ConvertApi\n  class ResultFile\n    def url\n      Addressable::URI.parse(info[\"Url\"]).normalize.to_s\n    end\n  end\nend\n```\nwhich requires this gem https://github.com/sporkmonger/addressable.\n\nRuby version 3.2.9 (but I think it does not depend)\nLibrary version 2.0.0 or 3.0.0 (tried on both, but I think it does not depend on library version, since the bug is in the API)\n\n","author":{"url":"https://github.com/DmitryBarskov","@type":"Person","name":"DmitryBarskov"},"datePublished":"2025-08-28T14:41:44.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/35/convertapi-library-ruby/issues/35"}