search

JustinBeckwith / yes-https

Say yes to https with express and connect.
MIT License
31 stars 2 forks source link

fix(lib): don't break App Engine Cron requests #2

Closed Splaktar closed 7 years ago

Splaktar commented 7 years ago

Tested in my project and this solved the problems mentioned in Slack.

Fixes #1

JustinBeckwith commented 7 years ago

Thanks for the PR! Instead of hard coding this stuff (including my awful _ah/health hack), I decided to just add an ignoreFilter function that lets you opt out of specific requests. See:

https://github.com/JustinBeckwith/yes-https#ignoring-specific-requests

Hope this helps!