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

Any way to enable secure API gateway? #179

Closed
mjgorman opened this issue Jul 11, 2016 · 6 comments
Closed

Any way to enable secure API gateway? #179

mjgorman opened this issue Jul 11, 2016 · 6 comments

Comments

@mjgorman
Copy link
Contributor

mjgorman commented Jul 11, 2016

If not, big feature request. Its not a deal breaker but having the ability in the zappa_settings.json to have the API Gateway create and produce an API key and require it on all the routes to secure things would be great.

@Miserlou
Copy link
Owner

Can you clarify this one a bit more?

To me, this seems like it should be an application-specific feature, not something that Zappa does, unless there is a specific Amazon service or API Gateway feature that you're talking about here.

What kind of API key? Just a password, or an OAuth scheme?

@mjgorman
Copy link
Contributor Author

Its an API Gateway feature. If you go into each of the API resources that get created there is an "Method Request" in Method Request, you can select to use an API Key, and Keys are generated in the API Gateway console. This gives you an x-api-key header you pass over to enable access control to the gateway.

@Miserlou
Copy link
Owner

Ahhhhhhhh. Okay, then yes, this is probably something that Zappa should handle. Do you have any links to docs/guides/tuts about this?

@mjgorman
Copy link
Contributor Author

Nothing crazy specific. If i remember correctly zappa is boto3, http://boto3.readthedocs.io/en/latest/reference/services/apigateway.html has methods for create_api_key/delete_api_key/get_api_key and most of the create_* method object returns have a key for 'apiKeyRequired', which just needs a boolean true.

@mjgorman
Copy link
Contributor Author

mjgorman commented Aug 2, 2016

#215 PR made.

@Miserlou
Copy link
Owner

Miserlou commented Aug 3, 2016

Merged! Thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants