Empty data attributes are stripped

[ryanb]

Is there a reason why empty data attributes are stripped?

Loofah.fragment('<div data-foo></div>').scrub!(:prune).to_s
 #=> "<div></div>"
Loofah.fragment('<div data-foo="bar"></div>').scrub!(:prune).to_s
 #=> "<div data-foo=\"bar\"></div>"

From what I can tell, empty data attributes are valid in HTML 5.

[flavorjones]

Hi @ryanb, thanks for reporting this. I agree, empty data-* attributes are valid and I think this behavior is just an oversight. Support for data-* attributes was added in #59 and expanded in #84 but at no point were empty attributes tested.

I think I can get to this in the next day or two, hang tight!

[flavorjones]

PR is #216 going through CI now. If it goes green, I'll ship it in v2.12.0 shortly thereafter.

[flavorjones]

v2.12.0 has been released with a fix: Release 2.12.0 / 2021-08-11 · flavorjones/loofah

[ryanb]

Wow thanks for the fast turnaround! I really appreciate the work you put into loofah.

[flavorjones]

Ah, thanks for the kind words.