fedora-hubs

#422 Sanitize text in Sticky widget
Merged by abompard. Opened by shaily.
shaily/fedora-hubs safe-md  into  develop

Download 422.patch

Fixes #421

rebased onto eb91b3526ec1a01daecaec39d6cf13864b53ab51

I'd love to see a unit test for that! :-)

Ok this is breaking the test I had written earlier for tables because it's stripping the <thead> tag :|

Now that the mechanism is in place, I'll try and find an exhaustive list of HTML elements that Markdown (and PHP Markdown Extra - the plugin we're using for additional functionality like tables) could possibly render.

WIP

2 new commits added

  • Add test for sanitized markdown output
  • Add method to update test widget configuration

3 new commits added

  • Add test for sanitized markdown output
  • Add method to update test widget configuration
  • Sanitize markdown in Sticky widget

1 new commit added

  • Update allowed tags

Not a whole lot of changes.
The tests pass now.

rebased onto 403fd0ff5be33b32be863dbf1d48607cd019307f

rebased onto 6847147e999c2e167fc2c63978b9f118db5b387d

Looks good, thanks!

Pull-Request has been merged by abompard
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.