Parse PUT requests and populate request.context from body content #441

Merged
merged 1 commit into from Jul 11, 2016

Conversation

Projects
None yet
2 participants
@aliaspooryorik
Contributor

aliaspooryorik commented Jul 7, 2016

Initial pass at supporting params in PUT requests.

Have tested on ACF 10 with Apache. Needs testing on Lucee / IIS and other CF versions which I haven't had a chance to do yet.

Have added the code to be conditional on enableJSONPOST being true so that it needs to be explicitly enabled to reduce potential impact on existing projects.

Feedback welcome!

@aliaspooryorik

This comment has been minimized.

Show comment
Hide comment
@aliaspooryorik

aliaspooryorik Jul 7, 2016

Owner

Initial commit - have tested on ACF 10 with Apache. Needs testing on Lucee / IIS and other CF versions.

Have added the code to be conditional on enableJSONPOST being true so that it needs to be explicitly enabled to reduce potential impact on existing projects.

Feedback welcome!

Owner

aliaspooryorik commented on f408646 Jul 7, 2016

Initial commit - have tested on ACF 10 with Apache. Needs testing on Lucee / IIS and other CF versions.

Have added the code to be conditional on enableJSONPOST being true so that it needs to be explicitly enabled to reduce potential impact on existing projects.

Feedback welcome!

@seancorfield seancorfield modified the milestone: 4.0 Jul 7, 2016

@seancorfield seancorfield merged commit e6ec36d into framework-one:develop Jul 11, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

seancorfield added a commit that referenced this pull request Jul 11, 2016

Minor code cleanup #441
Also add credit to John Whish for URL-encoded form data.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment