Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CSRF to graphql requests #39

Open
oliverzheng opened this issue Jun 4, 2018 · 5 comments
Open

Add CSRF to graphql requests #39

oliverzheng opened this issue Jun 4, 2018 · 5 comments

Comments

@oliverzheng
Copy link
Owner

No description provided.

@joshjcarrier
Copy link
Collaborator

You wanna do rate limit too?

@oliverzheng
Copy link
Owner Author

Sure why not. Not sure how to do that without adding stuff to the database though.

I think security has a higher priority of dealing with spam so we can add it later when/if we have a lot of traffic.

@joshjcarrier
Copy link
Collaborator

https://www.npmjs.com/package/lambda-rate-limiter Is a good simple start in case we accidentally dos ourselves

@oliverzheng
Copy link
Owner Author

Oooooo.

We should have just done npm install lgtmeme since there seems to be a package for everything.

@joshjcarrier
Copy link
Collaborator

You mean yarn add lgtmeme

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

No branches or pull requests

2 participants