| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
This fixes #20.
|
@developit Hey I'm using vhtml and quite liking it but I happened to look at some of these open pull requests and issues, and it looks like development has stalled a bit. There are outstanding problems I'd deem quite important to fix (#20, #34), and this pull request seems good to merge far as I can tell. There is also the fragment support which supposedly warrants some code changes, and a waiting MR (which also looks good to me) for that as well. (#31 (comment), #33) Is there something I could do to help? |
Sorry, something went wrong.
|
Wouldn't this be prone to the same issue you mentioned in #23 (comment)? The string might be evicted from the sanitized set before we use it, changing the output of vhtml(), returning escaped text instead of HTML. Example from the comment:
|
Sorry, something went wrong.
|
@johannesodland You are correct that this PR introduces more problems than it fixes. I forked this project here in order to fix the outstanding issues with this project. It might suit your use case. My impression is that vhtml is abandoned by @developit. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
This fixes #20.