-
Notifications
You must be signed in to change notification settings - Fork 38
set up a sandbox for security researchers #561
Comments
Definitely the right answer to gratipay/gratipay.com#3925. It should avoid the spam of fake teams and fake users, giving more details about the real usage of gratipay (I guess the statistics are somehow wrong due to theses trials). In addition, it should protect us against "wrong moves" ("Ooops, corrupted the database because of [some tool]"), DoS (it's out of scope but the report 123697 is a proof that some researchers are trying anyway)… Here are my thoughts about this:
What am I missing? |
Let's use a different domain entirely (to solidly avoid any potential cookie leaks), and not one with
Let's change the logo and banner, as well, for good measure, and also have a modal on first visit saying "Go away!" (suppressed for the next hour, maybe).
Gratipay.com is hosted on Heroku, where most of the relevant configuration is in the environment. One consideration is that, if we reproduce our production environment exactly, then it will double our hosting costs ($7/mo for one dyno, $20/mo for SSL, $50/mo for database). It's also possible that we may be able to implement this using Heroku's Pipelines feature. |
This is not supposed to happen but this article proves that it may still leak across subdomains, you're right. Don't take any risk and let's go for a
👍
Yep, my brain mixed the infrastructure of grtp.co and gratipay.com. If we want to provide SSL (so we won't have reports about missing HTTPS) for this instance, we can take a look at Let's encrypt. We can automate the renewal so we'll never have to care about it. Are Heroku's Pipelines introducing new costs for the review instances? I guess that yes :^) |
That seems fine for most security testing. SSL and DoS testing won't be entirely accurate. Seems like the next step here would be PRs for the UI and team review changes. Eh? |
This could help alleviate the accumulation of ~user and Team accounts we're seeing, e.g.
h/t https://hackerone.com/reports/128844#activity-893086:
The text was updated successfully, but these errors were encountered: