search

firstlane / buwwshit

https://firstlane.github.io/buwwshit
1 stars 0 forks source link

Can now handle HTML. Closes #16. #24

Closed firstlane closed 3 years ago

firstlane commented 3 years ago

This breaks the censoring feature. Need to fix that before this is merged.

firstlane commented 3 years ago

I did try testing this with a corporate email. I ran into a problem where the email had \<tr> and \<td> tags without an associated \<table> tag. Most browsers end up stripping these out, declaring them as stray tags, and it breaks the page when you view it. I don't know if all the emails we receive have this formatting issue or if it was just this one.

firstlane commented 3 years ago

I'm gonna merge this as is. I got carried away and fixed this while also working on another feature, and I don't feel like pulling the changes into separate commits. I'll put the censor fix up in another PR.

firstlane commented 3 years ago

Nevermind. It looks like I put the censoring code up in this PR.