Skip to content

Fix examples for ESP32SPI + SSL socket #58

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

Merged
merged 2 commits into from
Dec 30, 2020

Conversation

brentru
Copy link
Member

@brentru brentru commented Dec 30, 2020

Addresses #57 and fixes RuntimeError: Cannot access content after getting text or json error in examples/requests_advanced.py

@anecdata
Copy link
Member

Tested examples/requests_simpletest.py & examples/requests_advanced.py on PyPortal with 6.1.0-beta.2 and other libraries from 20201225. Also tested HTTPS. Looks good.

@brentru brentru requested a review from a team December 30, 2020 20:59
@brentru
Copy link
Member Author

brentru commented Dec 30, 2020

@anecdata sorry for the re-ping, thought i requested circuitpython librarians as a reviewer. Could you please re-review? Thanks

Copy link
Member

@anecdata anecdata left a comment

Choose a reason for hiding this comment

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

Tested and looks good (see comments above).

@anecdata
Copy link
Member

@brentru I'm kinda new to circuitpythonlibrarians ...what's the etiquette here: who merges? who releases?

@brentru
Copy link
Member Author

brentru commented Dec 30, 2020

@anecdata In my experience, CircuitPython Librarians generally review and merge/release (or @dherrada sometimes releases, I do not know the full process here) if the original PR author doesn't have access to the repo.

@brentru brentru merged commit 7cce618 into adafruit:master Dec 30, 2020
@brentru brentru deleted the fix-ssl-examples branch December 30, 2020 21:39
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Dec 31, 2020
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.

2 participants