-
Notifications
You must be signed in to change notification settings - Fork 43
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
Sync with upstream #62
base: master
Are you sure you want to change the base?
Conversation
Since there is no start condition interrupt
Limitations: - available only returns 1 or 0 - connect(ip, port) not implemented - trust anchors hard coded in file - requires _gettimeofday implementation in sketch
… is NULL This let’s us use it in Client::available()
By using br_sslio_read(ctx, NULL, len) hack
Used to return _gettimeofday value
…h if dst is NULL" This reverts commit 2bd615f380611fc26dd5ecdb916a91f90f9763ae.
This reverts commit 30b1db8865ac4cc6c5468f43ba7725378afab4e0.
…on Arduino" This reverts commit 0d353757d75a2c83129d220bf3dc731e08a2669a.
This reverts commit 6f82133.
… closed when connection ends
Fix issue due to repeated status()/connected() call
* Nano RP2040 also requires the `no_reset` flag.
Ninafw bug fixes
support server.accept() for WiFiNINA library
Add a new command to return the DNS IP.
# Conflicts: # CHANGELOG # README.md # arduino/cores/esp32/wiring_digital.h # arduino/libraries/WiFi/src/WiFi.cpp # arduino/libraries/WiFi/src/WiFi.h # arduino/libraries/WiFi/src/WiFiClient.cpp # combine.py # data/roots.pem # main/CommandHandler.cpp # main/sketch.ino.cpp # sdkconfig
@hathach I didn't look at this carefully enough. Are you waiting for us to test this? You mentioned it's not working with that Let's Encrypt site. It's using the certs submodule now, right? |
@dhalbert yeah, I was waiting for your feedback on wip port before making more changes to address the encrypted site issue. Then got switched to other works, and kind of forgot about this issue. I will try to put it back for more tests. |
@dhalbert so I just tried this on a New
Current
|
sync with upstream, try to keep as close as upstream as possible while reserved as much modification as possible (comparing to upstream arduino/nina-fw@master...adafruit:nina-fw:master). File changes may look a lot (376) but most of them are new file addtion. Also I don't know why PR commit start with initial one (weird).
I only take a few examples from WiFiNINA: WiFiSSLClient, WiFiWebClient. @danh @brentru @ladyada please help to review and test this out. If you have any sophisticated that test most of nina function, please let me know (I am not too familliar with nina). Once merged, we can start another PR to upgrade IDF from v3 to v5
PS: just tested with
valid-isrgrootx2.letsencrypt.org
(previously fixed by #57) and got -0x2700 (MBEDTLS_ERR_X509_CERT_VERIFY_FAILED). Will try to fix and more revise later.PS2: ci is updated to also build with debug message ON, uploaded as
NINA_W102-*_debug.bin
in debug matrix for testing convenience.