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

Connection successful even if password provided is wrong? #356

Open
DavideViolante opened this issue Jan 15, 2024 · 10 comments
Open

Connection successful even if password provided is wrong? #356

DavideViolante opened this issue Jan 15, 2024 · 10 comments

Comments

@DavideViolante
Copy link
Collaborator

DavideViolante commented Jan 15, 2024

I'm trying to connect to the SSID and PSW provided by the user input (see wifiConfigDevice const):

    try {
      const wifiConfigDevice: DeviceWifiConfig = { wifiSsid, wifiPwd };
      // Force connection on the Wi-Fi the user provided
      await WifiManager.connectToProtectedSSID(wifiSsid, wifiPwd, false, false);
      ...
    } catch (error) {
      ...
      failToast(intl.formatMessage({ id: 'deviceStartupFail' }));
    }

After this code, even if the user types a wrong password, the connection is successful.
Why is this happening?
Is there any way to force the check of SSID and PSW of a WiFi to be sure the user types the correct credentials?
I need to be sure the credentials are correct since we are saving those into a IoT device after this step.

Video on Android SDK 29 (Android 10):
https://github.com/JuanSeBestia/react-native-wifi-reborn/assets/6844116/89bee5e3-7d44-459a-861d-10a6afae8950
In the video, the "ciaociaociao" psw is wrong for that network.
Also if I'm not already connected, the psw check is skipped.

The only way it seems to work is when the phone never connected to that network before (forget network) and I try the first time using the library. In that case the psw is checked. But I need the psw to be checked always.

Thanks

@DavideViolante DavideViolante changed the title Connection successfully even if password provided is wrong? Connection successful even if password provided is wrong? Jan 15, 2024
Copy link

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community attention? This issue may be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Feb 15, 2024
@DavideViolante
Copy link
Collaborator Author

Hello, any news about this @JuanSeBestia ?
Thank you

@JuanSeBestia
Copy link
Owner

Feel free to try to fix it, then open a MR ❤️

@DavideViolante
Copy link
Collaborator Author

I wanted to be sure that it's a real bug or I'm doing something wrong. Anyone else experiecing the same issue? Can you reproduce it too?

@github-actions github-actions bot removed the stale label Feb 22, 2024
@wtlyu
Copy link

wtlyu commented Feb 23, 2024

Yes. same issues here.

Copy link

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community attention? This issue may be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Mar 29, 2024
@DavideViolante
Copy link
Collaborator Author

This should not be closed at all.

@github-actions github-actions bot removed the stale label Apr 1, 2024
Copy link

github-actions bot commented May 1, 2024

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community attention? This issue may be closed if no further activity occurs.

@github-actions github-actions bot added the stale label May 1, 2024
@github-actions github-actions bot closed this as completed May 8, 2024
@JuanSeBestia JuanSeBestia reopened this May 28, 2024
@DavideViolante
Copy link
Collaborator Author

@alexma01 Hello Ale, do you think you also fixed this issue in your PR #355 ?

@github-actions github-actions bot removed the stale label May 29, 2024
Copy link

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community attention? This issue may be closed if no further activity occurs.

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

No branches or pull requests

3 participants