resolves #4461 wrap target of link used in place of include directive…

… in inline passthrough when it contains spaces (PR #4463)

CHANGELOG.adoc

@@ -62,6 +62,10 @@ Improvements::
   * Add single and double role hint to `<quote>` tag in DocBook output (#2947)
   * Use safe navigation to avoid crashing when querying for extensions
 
+Bug Fixes::
+
+  * Escape spaces in include target (using inline passthrough) when generating link from include directive (#4461)
+
 == 2.0.20 (2023-05-18) - @mojavelinux
 
 Bug Fixes::

lib/asciidoctor/reader.rb

@@ -1026,6 +1026,7 @@ def preprocess_include_directive target, attrlist
     # if running in SafeMode::SECURE or greater, don't process this directive
     # however, be friendly and at least make it a link to the source document
     elsif doc.safe >= SafeMode::SECURE
+      expanded_target = %(pass:c[#{expanded_target}]) if expanded_target.include? ' '
       # FIXME we don't want to use a link macro if we are in a verbatim context
       replace_next_line %(link:#{expanded_target}[role=include#{attrlist ? ',' + attrlist : ''}])
     elsif @maxdepth
@@ -1227,6 +1228,7 @@ def resolve_include_path target, attrlist, attributes
     if (Helpers.uriish? target) || (::String === @dir ? nil : (target = %(#{@dir}/#{target})))
       unless doc.attr? 'allow-uri-read'
         logger.warn message_with_context %(cannot include contents of URI: #{target} (allow-uri-read attribute not enabled)), source_location: cursor
+        target = %(pass:c[#{target}]) if target.include? ' '
         return replace_next_line %(link:#{target}[role=include#{attrlist ? ',' + attrlist : ''}])
       end
       if doc.attr? 'cache-uri'

test/reader_test.rb

@@ -677,6 +677,13 @@ class ReaderTest < Minitest::Test
         assert_equal 'link:include-file.adoc[role=include]', reader.read_line
       end
 
+      test 'should escape spaces in target when generating link from include directive' do
+        input = 'include::foo bar baz.adoc[]'
+        doc = Asciidoctor::Document.new input
+        reader = doc.reader
+        assert_equal 'link:pass:c[foo bar baz.adoc][role=include]', reader.read_line
+      end
+
       test 'should preserve attrlist when replacing include directive with link macro' do
         input = 'include::include-file.adoc[leveloffset=+1]'
         doc = Asciidoctor::Document.new input
@@ -694,6 +701,16 @@ class ReaderTest < Minitest::Test
         end
       end
 
+      test 'should escape spaces in target when generating link from remote include directive' do
+        using_memory_logger do |logger|
+          input = 'include::https://example.org/no such file.adoc[]'
+          doc = Asciidoctor::Document.new input, safe: :safe
+          reader = doc.reader
+          assert_equal 'link:pass:c[https://example.org/no such file.adoc][role=include]', reader.read_line
+          assert_message logger, :WARN, '<stdin>: line 1: cannot include contents of URI: https://example.org/no such file.adoc (allow-uri-read attribute not enabled)', Hash
+        end
+      end
+
       test 'should preserve attrlist when replacing remove include directive with link macro' do
         using_memory_logger do |logger|
           input = 'include::https://example.org/dist/info.adoc[leveloffset=+1]'