lirantal / lirantal/essential-nodejs-security-book
Some feedback
Open
@lirantal is already working on this.
Since Jan 8, 2017.
- Dominant language
- JavaScript
- Stars
- 96
- Forks
- 35
- PR merge metrics
- No merged PRs in 30d
Description
Overall, very solid. I love the details you've provided in the clear and concise matter.
Some thing I would like to see discussed:
- CORS (is there anything wrong with it?)
- JWT (at least from an overview stand point)
Do you want PRs made for minor grammatical mistakes?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.