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.

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

samples: sockets: http_get: More configurability and logging #5981

Conversation

pfalcon
Copy link
Contributor

@pfalcon pfalcon commented Feb 5, 2018

Make it easy to override HTTP host/port/path. Print URL which we
request and make few other adjustment to the output for clarity.

Signed-off-by: Paul Sokolovsky paul.sokolovsky@linaro.org

@pfalcon pfalcon requested review from jukkar and GAnthony February 5, 2018 14:08
@pfalcon
Copy link
Contributor Author

pfalcon commented Feb 5, 2018

I'd like to use this sample as a guinea ping for #5900 , makes sense to clean it up first.

@pfalcon pfalcon added this to the v1.11.0 milestone Feb 5, 2018
@codecov-io
Copy link

codecov-io commented Feb 5, 2018

Codecov Report

Merging #5981 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5981   +/-   ##
=======================================
  Coverage   52.51%   52.51%           
=======================================
  Files         406      406           
  Lines       39682    39682           
  Branches     7715     7715           
=======================================
  Hits        20838    20838           
  Misses      15663    15663           
  Partials     3181     3181

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 63f862b...c09bd7a. Read the comment docs.

@pfalcon pfalcon force-pushed the net-samples-sockets-http_get-better branch from 828866c to 8398756 Compare February 5, 2018 14:28
Make it easy to override HTTP host/port/path. Print URL which we
request and make few other adjustment to the output for clarity.

Signed-off-by: Paul Sokolovsky <paul.sokolovsky@linaro.org>
Copy link
Collaborator

@GAnthony GAnthony left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pfalcon pfalcon force-pushed the net-samples-sockets-http_get-better branch from 8398756 to 19d9231 Compare February 5, 2018 17:20
@pfalcon pfalcon requested a review from tbursztyka as a code owner February 5, 2018 17:20
@pfalcon pfalcon force-pushed the net-samples-sockets-http_get-better branch from 19d9231 to c09bd7a Compare February 5, 2018 17:22
@jukkar jukkar merged commit 166cd7f into zephyrproject-rtos:master Feb 6, 2018
@pfalcon pfalcon deleted the net-samples-sockets-http_get-better branch September 6, 2018 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants