Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

fixes #209 - replaced accept-whitelist and content-type-whitelist #214

Closed
wants to merge 1 commit into from
Closed

fixes #209 - replaced accept-whitelist and content-type-whitelist #214

wants to merge 1 commit into from

Conversation

vik-singh
Copy link

fixes #209 - replaced accept-whitelist and content-type-whitelist with their underscored counterparts since the service now does not ignore invalid keys in config

…h their underscored counterparts since the service now does not ignore invalid keys in config
@vik-singh
Copy link
Author

@TomHAnderson - thanks just saw you included these changes in yours - closing mine!

@vik-singh vik-singh closed this Jul 16, 2015
@TomHAnderson
Copy link
Contributor

This is a good effort. Technically a patch is labeled with prefix hotfix such as hotfix/209-fix-config-dash

You were doomed to fail because the content negotiation is more specific now and changes were required to the tests, included in #213

@vik-singh vik-singh deleted the fix-issue-209 branch July 16, 2015 04:25
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.

accept-whitelist instead of accept_whitelist in config is causing error
2 participants