-
Notifications
You must be signed in to change notification settings - Fork 212
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
Always enfore strict header checking [broken] #174
Conversation
This forces all kinds of upgrades on people, like bluebird, alexa-utterances, etc., while the library is perfectly compatible with older verisons. I think you should revert package.json and only do the upgrade that is required by the interface change. |
Also build seems to have broken because of the danger upgrade. |
Still broken, revert package.json and do only the update of the minimum version of alexa-verifier-middleware. |
@@ -183,15 +183,6 @@ describe("Alexa", function() { | |||
}); | |||
}); | |||
|
|||
it("requires a cert header", function() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason for removing this test?
This will be removed in the next major version
Also updated logging of old functionality to include stack trace
Updated
alexa-verifier-middleware
to0.2.1
from0.1.9
, which removes thestrictHeaderCheck
option. Either the verification should be completely on, or completely off.