go-premailer

go-premailer copied to clipboard

I added a test case that breaks when you upgrade golang.org/x/net, which broke the Hermes e-mail templating package downstream.

This requires a downstream update of goquery, and /x/net/http. I believe that /x/net/http was resolved, but Goquery hasn't upgraded yet https://github.com/PuerkitoBio/goquery/issues/391. After it has, the version here should be bumped too.

Seeing the same issue here, where conditional comments are broken:

Old output

<!--[if mso]>\n      <style type=\"text/css\">\n...

New (bad) output

<!--[if mso]&gt;\n      &lt;style type=&#34;text/css&#34;&gt;\n...

Can anyone help me following those issues and upgrades? thanks

After additional digging, this issue is not resolved in x/net. I've added a note to https://github.com/golang/go/issues/46261 indicating where the bug was introduced.

dependabot upgraded golang.org/x/net https://github.com/vanng822/go-premailer/pull/20 and it seems work fine?

Look like tests passed if I upgraded golang.org/x/net to v0.23.0. Could you try it out then I can merge this instead of of #20

Look like it works fine so I will merge.