Skip to content

Conversation

@namebee
Copy link

@namebee namebee commented Nov 24, 2024

I've added upon to the benchmark as I have been considering it as an option for one of my projects and I could not find a good speed comparison to other engines.

@Kijewski
Copy link
Member

Thank you for your PR! Your implementation looks good, but I have a small problem with the library itself.

Automatic HTML escaping is, at least for me, an essential feature, and our readme also mentions that all tested libraries do HTML escaping by default. upon does not. To me, this is too big a security risk to promote the library. Once auto-HTML-escaping becomes configurable, I'd be happy to add it to the list!

@Kijewski Kijewski force-pushed the main branch 3 times, most recently from dd71a26 to d3c1b50 Compare December 27, 2024 16:46
@Kijewski Kijewski force-pushed the main branch 6 times, most recently from c1332fd to fc6a734 Compare May 16, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants