Skip to content

Small improvements on HTTPUpdate #2156

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Dec 6, 2018
Merged

Small improvements on HTTPUpdate #2156

merged 1 commit into from
Dec 6, 2018

Conversation

Jeroen88
Copy link
Contributor

@Jeroen88 Jeroen88 commented Dec 5, 2018

Abort update if http.begin() returns false.

Fix a typo in httpUpdate.ino

@Jeroen88 Jeroen88 mentioned this pull request Dec 5, 2018
Merged
@me-no-dev
Copy link
Member

error checking FTW!!! :) Thanks!

@me-no-dev me-no-dev merged commit 0596a2a into espressif:master Dec 6, 2018
@Jeroen88
Copy link
Contributor Author

Jeroen88 commented Dec 6, 2018

:-D

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
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
@Jeroen88 @me-no-dev