Closing slashes in single tags (img, br)

[l3x4]

There's an example use case here: http://wonko.com/post/sanitize
It's actually very close to the Readme example, but differs a bit.

html = '<b><a href="http://foo.com/">foo</a></b><img src="http://foo.com/bar.jpg" />'
Sanitize.clean(html, Sanitize::Config::RELAXED)
# => '<b><a href="http://foo.com/">foo</a></b><img src="http://foo.com/bar.jpg" />'

Please note the closing slash on the image tag - that's what I would actually expect as the result.

In reality the same example cuts out that slash:

html = '<b><a href="http://foo.com/">foo</a></b><img src="http://foo.com/bar.jpg" />'
Sanitize.clean(html, Sanitize::Config::RELAXED)
#=> '<b><a href="http://foo.com/">foo</a></b><img src="http://foo.com/bar.jpg">'

Same happens to slashes in br tags for example.
Is there a way to keep these slashes?

Thank you.

[rgrove]

That blog post was the initial announcement of Sanitize, back when Sanitize's output was XHTML by default. Since version 2.0, the default output format has been HTML. To preserve XHTML self-closing tags, just set the :output config option to :xhtml.

[l3x4]

Thank you very much for the help and for this very useful library! :)

[l3x4]

But wait a sec, I still don't see the expected behaviour here.

html = '<b><a href="http://foo.com/">foo</a></b><img src="http://foo.com/bar.jpg" />'
Sanitize.clean(html,Sanitize::Config::RELAXED.merge({:output => :xhtml}))
#=> "<b>\n<a href=\"http://foo.com/\">foo</a>\n</b><img src=\"http://foo.com/bar.jpg\" />"

Now there are \ns before and after the a tag

[rgrove]

I can't reproduce this. I get the expected output:

#=> "<b><a href=\"http://foo.com/\">foo</a></b><img src=\"http://foo.com/bar.jpg\" />"

What version of Ruby and Nokogiri do you have installed?

[l3x4]

I use ree-1.8.7-head (via rvm) and nokogiri-1.5.0

[rgrove]

You're right. With Ruby 1.8.7 and Nokogiri 1.5.0, I get extra newlines. This definitely isn't anything Sanitize is doing though. It also happens when I call Nokogiri directly:

>> Nokogiri::HTML.fragment('<b><a href="http://foo.com/">foo</a></b><img src="http://foo.com/bar.jpg" />').to_xhtml
=> "<b>\n  <a href=\"http://foo.com/\">foo</a>\n</b><img src=\"http://foo.com/bar.jpg\" />"

You may want to file a bug against Nokogiri.

[l3x4]

Thank you!
I've raised this with Nokogiri: https://github.com/tenderlove/nokogiri/issues/543