Skip to content
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

.begin() for WEP is in Wifi.h but is not implemented in Wifi.cpp. #819

Closed
K1ttyKat opened this issue Jan 6, 2024 · 1 comment · Fixed by #874
Closed

.begin() for WEP is in Wifi.h but is not implemented in Wifi.cpp. #819

K1ttyKat opened this issue Jan 6, 2024 · 1 comment · Fixed by #874

Comments

@K1ttyKat
Copy link

K1ttyKat commented Jan 6, 2024

I have been asked to raise an issue here but I have never done this before so forgive any errors in formatting, etc.

I have a Giga R1 and the accompanying display and wanted to get the time from an NTP server. I therefore followed the official Arduino examples shown here: https://docs.arduino.cc/tutorials/giga-r1-wifi/giga-wifi

I found that I could make a WPA connection using...

status = WiFi.begin(ssid, pass);

...but that when I tried to make a WEP connection by adding the network index number like this:

status = WiFi.begin(ssid, keyIndex, pass);

...I got a compilation error.

I raised this on the Arduino Forum here https://forum.arduino.cc/t/giga-wifi-not-working/1207373 and a helpful person called stevebeswick replied saying that "the .begin() for WEP is in Wifi.h but is not implemented in Wifi.cpp" and suggested that I should raise an issue here about it - which I have now done.

I hope this provides all necessary information.

@facchinm
Copy link
Member

facchinm commented Jan 9, 2024

@K1ttyKat what happens if you try to connect to the WEP network using the WiFi.begin(ssid, pass); API? The undelying mbed-os WiFi API doesn't handle WEP keys explicitly (https://os.mbed.com/docs/mbed-os/v6.16/apis/wi-fi.html) but it automatically handle the key index.
If it works, I'd add a fake WiFi.begin(ssid, keyIndex, pass); implementation to the library ignoring the second paramenter.

iabdalkader added a commit to iabdalkader/ArduinoCore-mbed that referenced this issue Apr 22, 2024
- Add support for connecting to hidden networks (issue arduino#855).
- Implement `begin()` for WEP (issue arduino#819).
- Add sanity checks to all functions accepting network index from the user.

Signed-off-by: iabdalkader <[email protected]>
iabdalkader added a commit to iabdalkader/ArduinoCore-mbed that referenced this issue Apr 24, 2024
- Add support for connecting to hidden networks (issue arduino#855).
- Implement `begin()` for WEP (issue arduino#819).
- Add sanity checks to all functions accepting network index from the user.

Signed-off-by: iabdalkader <[email protected]>
iabdalkader added a commit to iabdalkader/ArduinoCore-mbed that referenced this issue Apr 24, 2024
- Add support for connecting to hidden networks (issue arduino#855).
- Implement `begin()` for WEP (issue arduino#819).
- Add sanity checks to all functions accepting network index from the user.

Signed-off-by: iabdalkader <[email protected]>
iabdalkader added a commit to iabdalkader/ArduinoCore-mbed that referenced this issue Apr 25, 2024
- Add support for connecting to hidden networks (issue arduino#855).
- Implement `begin()` for OPEN security.
- Deprecate WEP security (issue arduino#819).
- Add sanity checks to all functions accepting network index from the user.

Signed-off-by: iabdalkader <[email protected]>
multiplemonomials pushed a commit to mbed-ce/ArduinoCore-mbed-ce that referenced this issue Jun 23, 2024
- Add support for connecting to hidden networks (issue arduino#855).
- Implement `begin()` for OPEN security.
- Deprecate WEP security (issue arduino#819).
- Add sanity checks to all functions accepting network index from the user.

Signed-off-by: iabdalkader <[email protected]>
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 a pull request may close this issue.

2 participants