search

85x14 / griddler-ses

MIT License
13 stars 29 forks source link

Remove naive middleware check for a griddler path #8

Open jdurand opened 7 years ago

jdurand commented 7 years ago
  1. Mounting Griddler is not a requirement: https://github.com/thoughtbot/griddler#installation
  2. The check didn't take domains/subdomains into account
  3. Checking for the HTTP_X_AMZ_SNS_MESSAGE_TYPE header is sufficient
  4. This actually breaks if Griddler is not mounted