Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add ability to get raw http request body + Example #499

Closed
wants to merge 1 commit into from
Closed

Add ability to get raw http request body + Example #499

wants to merge 1 commit into from

Conversation

avr39-ripe
Copy link
Contributor

Squashed version of #487 + Example

@avr39-ripe avr39-ripe mentioned this pull request Dec 23, 2015
Closed
@avr39-ripe
Copy link
Contributor Author

@hreintke this is squashed version as you requested
in addition there is example to demonstrate this and several other features and concepts, I try to make example small, simple, but reusable.
it fully obsolets #272 (but I mention again to be honest, I'm not the author of this feature, I just gather and clean things)
I test it and it works on real hardware.

@r1m
Copy link

r1m commented Jan 10, 2016

Pretty usefull PR.
Side notes :

  • example eclipse project is named MeteoControl which collides with the real one
  • all examples moved to a new folder since 071d2d5

@avr39-ripe
* Add ability to get raw http request body + BasicWebSkeletonApp as e…
653ea75 
…xample

* Add response.getBody() to get raw http request body if content type is other than FormUrlEncoded
* Add Simple Skeleton app to be used as starting point for some useful app demonstrating several nice concepts

* First attempt to get request body out from server

* bit polish

* Move parseRawBody() bit upper and return ERR_OK after it to speed thing up

* Add BasicWebSkeletonApp
@r1m r1m mentioned this pull request Jan 16, 2016
Closed
@avr39-ripe avr39-ripe force-pushed the feature/http_raw_body branch from 0857d58 to 653ea75 Compare January 16, 2016 12:34
@hreintke
Copy link
Contributor

@avr39-ripe :
I merged the core functionality in PR #578.
Left out the example as now

  • Should go to the samples directory
  • Need generalization + it now overlaps with MeteoControl application.

@hreintke hreintke closed this Jan 31, 2016
@hreintke hreintke added this to the 3.0 milestone Jan 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Enhancement
Projects
None yet
Milestone
3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@avr39-ripe @r1m @hreintke