Skip to content

Porting of ArduinoHttpClient's Examples for MKRGSM #47

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
Closed
wants to merge 0 commits into from
Closed

Porting of ArduinoHttpClient's Examples for MKRGSM #47

wants to merge 0 commits into from

Conversation

Rocketct
Copy link

@Rocketct Rocketct commented Nov 9, 2018

  • Changed the file system examples structure;
  • Modified the examples for allow the connection with the MKRGSM board;
  • Removed all the unnecessary examples;

i have change also the server URL in the examples to httpbin.org where required (get, post put, delete ...), a typical successful output is the following:

Starting Arduino web client.
connected
making DELETE request
Status code: 200
Response: {
  "args": {}, 
  "data": "", 
  "files": {}, 
  "form": {
    "age": "46", 
    "name": "light"
  }, 
  "headers": {
    "Connection": "close", 
    "Content-Length": "17", 
    "Content-Type": "application/x-www-form-urlencoded", 
    "Host": "httpbin.org", 
    "User-Agent": "Arduino/2.2.0"
  }, 
  "json": null, 
  "origin": "185.3.54.12", 
"url": "http://httpbin.org/delete"

}

Wait five seconds

let me know if ok for you.
cc/ @sandeepmistry @facchinm @akash73

sandeepmistry
sandeepmistry reviewed Nov 9, 2018
View reviewed changes
Copy link

@sandeepmistry sandeepmistry left a comment

Choose a reason for hiding this comment

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

Since the GSM files use a public web server, should we consider to use the same in the WiFi examples and remove the Node.js test server?

examples/WIFI/CustomHeader/CustomHeader.ino Outdated
client.write((const byte*)postData.c_str(), postData.length());
client.endRequest();
Copy link

Choose a reason for hiding this comment

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

Are you sure about this change?

@Rocketct Rocketct closed this Feb 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sandeepmistry sandeepmistry sandeepmistry left review comments

@akash73 akash73 Awaiting requested review from akash73

@facchinm facchinm Awaiting requested review from facchinm

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Rocketct @sandeepmistry