-
Notifications
You must be signed in to change notification settings - Fork 86
ESP8266 support for Arduino IoT Cloud (User/Pass Authentification) #76
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
Conversation
This comment has been minimized.
This comment has been minimized.
bc6843c to
3275bff
Compare
This comment has been minimized.
This comment has been minimized.
a3b08b0 to
c32fcad
Compare
This comment has been minimized.
This comment has been minimized.
|
I don't know how the codacy/PR quality review was added to the CI build (maybe @per1234 or @gvarisco can shed some light on that) but I strongly suggest to remove it. I've got no intention of working through the useless formatting suggestions for |
458ea24 to
c32fcad
Compare
c8c6bef to
2a5ed56
Compare
This comment has been minimized.
This comment has been minimized.
Removed it. |
This comment has been minimized.
This comment has been minimized.
2a5ed56 to
42aed61
Compare
This comment has been minimized.
This comment has been minimized.
e943d5e to
9b70fbe
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Branch successfully tested with ESP8266 based board and MKR1000. @ubidefeo Can you please try it out with a MKRGSM1400? |
… now handled by 'Arduino_ConnectionHandler' and 'Arduino_DebugUtils'
…l function for ArduinoCloudBearSSL. The ArduinoCloudBearSSL function for getting the time is set within 'ArduinoIoTCloud::begin' already.
…ble for the static non-member 'getTime' function by exploiting the fact that ArduinoIoTCloudClass is anyway a global variable within the file
…information for user/pass authentification
…lass::setIoTConnectionState' since it served mostly the purpose of printing debug messages
…to 'SECRET_DEVICE_KEY'
…setSecretDeviceKey'
…ficate without verification and therefore susceptible to MITM attacks. This has been a mgmt decision since the other option would have been that the ESPs will not be able to connect to the ArduinoIoTCloud anymore when the leaf certificate is exchanged (which happens once/year or even sooner than that). Unfortunately the ESP8266 does not have the capability to verify the whole chain of trust which is the reason why the verification of the leaf certificate has been the option used in the first place - despite breaking minimum once/year.
e77a048 to
6d0d5ed
Compare
… now handled by 'Arduino_ConnectionHandler' and 'Arduino_DebugUtils'
…L upon construction
Just tested with a GSM 1400 and a NANO 33 IoT, it works |
|
I tested the example sketches used in the Getting Started and it worked very well, but there is just a little problem on my side: the message printed on the serial monitor when the board connects to the cloud has changed from |
|
Here is a list of the strings I'm checking during the Getting Started:
|
|
Hi @AlbyIanna 👋 sorry for messing with your string evaluation, I was not aware you are using that to determine system state. I've checked the strings you've provided in the above comment against the current implementation:
|
|
Rebased the changes of branch |
…l baudrate with the other example sketches
Travis tests have failedHey @lxrobotics, TravisBuddy Request Identifier: 22aa9f00-e365-11e9-b752-11f331f20c85 |
This pull request adds support for the ESP8266 to be used with the Arduino IoT Cloud utilizing user/password authentification with the cloud.
If you want to try this feature out you must follow the following steps:
Arduino_ConnectionHandlerwhich contains the necessary changes to support ESP8266 user/pass logingit clone https://github.com/arduino-libraries/Arduino_ConnectionHandler -b esp8266-user-pass-loginexamples/ArduinoIoTCloud_ESP8266ARDUINO_CLOUD_DEVICE_ID,ARDUINO_CLOUD_DEVICE_PASSandTHING_IDas well as your WIFI dataSECRET_SSIDandSECRET_PASSRequires arduino-libraries/Arduino_ConnectionHandler#1