Skip to content

Create SECURITY.md - #664

Merged
mde merged 3 commits into
mde:mainfrom
netcode:patch-1
Apr 23, 2022
Merged

Create SECURITY.md#664
mde merged 3 commits into
mde:mainfrom
netcode:patch-1

Conversation

@netcode

@netcode netcode commented Apr 10, 2022

Copy link
Copy Markdown
Contributor

Adding a basic security policy. Highly inspired by ExpressJS security policy.

Highly inspired by ExpressJS security policy.
@mde

mde commented Apr 11, 2022

Copy link
Copy Markdown
Owner

Could you please add a statement similar to the following?

"If you give end-users unfettered access to the EJS render method, you are using EJS in an inherently un-secure way. Please do not report security issues that stem from doing that. EJS is effectively a JavaScript runtime. Its entire job is to execute JavaScript. If you run the EJS render method without checking the inputs yourself, you are responsible for the results."

I ask this because we get a ton of supposed security issues that stem from people assuming the render method should be secure. It is not, and nor should it be.

netcode added 2 commits April 13, 2022 04:46
Update the readme to add the security note
@netcode

netcode commented Apr 23, 2022

Copy link
Copy Markdown
Contributor Author

Done, I added this note in the security.md & the readme.md

@mde
mde merged commit 66f7471 into mde:main Apr 23, 2022
@mde

mde commented Apr 23, 2022

Copy link
Copy Markdown
Owner

This is awesome, thank you so much!

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