Skip to content
This repository was archived by the owner on Feb 21, 2019. It is now read-only.

Support for Google App Engine - #38

Open
pavelkraleu wants to merge 1 commit into
not-kennethreitz:masterfrom
pavelkraleu:master
Open

Support for Google App Engine#38
pavelkraleu wants to merge 1 commit into
not-kennethreitz:masterfrom
pavelkraleu:master

Conversation

@pavelkraleu

Copy link
Copy Markdown

Backend tasks shouldn’t be redirected to SSL

Backend tasks shouldn’t be redirected to SSL
@pavelkraleu

Copy link
Copy Markdown
Author

These are App engine headers added only to backend (cron) tasks
https://cloud.google.com/appengine/docs/python/taskqueue/overview-push?hl=en#task_request_headers
https://cloud.google.com/appengine/docs/python/config/cron?hl=en#Python_app_yaml_Securing_URLs_for_cron

@mritchie712

mritchie712 commented Nov 10, 2017

Copy link
Copy Markdown

is this going to be merged? Seems like a straight forward change.

Edit - This works fine for my use case sslify = SSLify(app, skips=['mypath', 'anotherpath']) since my cron jobs only hit end points that aren't public

@Projjol-zz

Copy link
Copy Markdown
Member

@pavelkraleu if you'd resolve the merge conflicts, I can go ahead and merge your changes, thanks 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants