Skip to content

Commit 431ca52

Browse files
authored
Merge pull request FallibleInc#2 from eryno/patch-1
Minor copyediting on security checklist
2 parents cf7db39 + aec03ea commit 431ca52

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

security-checklist.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
- [ ] Destory all active sessions on reset password (or offer to).
1111
- [ ] Must have the `state` parameter in OAuth2
1212
- [ ] No open redirects after successful login or in any other intermediate redirects.
13-
- [ ] While Signup/Login input, sanitize input for javascript://, data://, CRLF characters.
13+
- [ ] When parsing Signup/Login input, sanitize for javascript://, data://, CRLF characters.
1414
- [ ] Set secure, httpOnly cookies.
1515
- [ ] In Mobile `OTP` based mobile verification, do not send the OTP back in the response when `generate OTP` or `Resend OTP` API is called.
1616
- [ ] Limit attempts to `Login`, `Verify OTP`, `Resend OTP` and `generate OTP` APIs for a particular user. Have an exponential backoff set or/and something like a captcha based challenge.
@@ -27,7 +27,7 @@
2727
- [ ] Any upload feature should sanitize the filename provided by the user. Also, for generally reasons apart from security, upload to something like S3 (and post-process using lambda) and not your own server capable of executing code.
2828
- [ ] `Profile photo upload` feature should sanitize all the `EXIF` tags also if not required.
2929
- [ ] For user ids and other ids, use [RFC complaint ](http://www.ietf.org/rfc/rfc4122.txt) `UUID` instead of integers. You can find an implementation for this for your language on Github.
30-
- [ ] JWT are awesome, use them if required for your single page app/APIs.
30+
- [ ] [JWT](https://jwt.io/) are awesome. Use them if required for your single page app/APIs.
3131

3232

3333
##### ANDRIOD / IOS APP
@@ -63,12 +63,12 @@
6363
- [ ] Use a decent provisioning script to create VMs in the cloud.
6464
- [ ] Check for machines with unwanted publicly `open ports`.
6565
- [ ] Check for no/default passwords for `databases` especially MongoDB & Redis. BTW MongoDB sucks, avoid it.
66-
- [ ] Use SSH to access your machines, do not setup a password.
66+
- [ ] Use SSH to access your machines; do not setup a password.
6767
- [ ] Install updates timely to act upon zero day vulnerabilities like Heartbleed, Shellshock.
6868
- [ ] Modify server config to use TLS 1.2 for HTTPS and disable all other schemes. (The tradeoff is good)
6969
- [ ] Do not leave the DEBUG mode on. In some frameworks, DEBUG mode can give access full-fledged REPL or shells or expose critical data in error messages stacktraces.
7070
- [ ] Be prepared for bad actors & DDOS - use [Cloudflare](https://www.cloudflare.com/ddos/)
71-
- [ ] Setup monitoring for your systems and log stuff (use Newrelic or something like that)
71+
- [ ] Setup monitoring for your systems and log stuff (use [New Relic](https://newrelic.com/) or something like that)
7272
- [ ] If developing for enterprise customers, adhere to compliance requirements. If AWS S3, consider using the feature to [encrypt data](http://docs.aws.amazon.com/AmazonS3/latest/dev/UsingServerSideEncryption.html). If using AWS EC2, consider using the feature to use encrypted volumes (even boot volumes can be encypted now).
7373

7474
##### PEOPLE
@@ -77,4 +77,4 @@
7777
- [ ] Be polite to bug reporters.
7878
- [ ] Have your code review done by a fellow developer from a secure coding perspective. (More eyes)
7979
- [ ] In case of a hack or data breach, check previous logs for data access, ask people to change passwords. You might require an audit by external agencies depending on where you are incorporated.
80-
- [ ] Setup Netflix Scumblr to hear about talks about your organization on social platforms and Google search.
80+
- [ ] Setup [Netflix's Scumblr](https://github.com/Netflix/Scumblr) to hear about talks about your organization on social platforms and Google search.

0 commit comments

Comments
 (0)