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

[WiFiSetting] Fixed an issue where SSID and PASSWD were not printed correctly if they exceeded 14 characters. #153

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sssscccc210
Copy link

@sssscccc210 sssscccc210 commented Sep 19, 2024

SSID and PASSWD were printed correctly when they were under 13 characters, but they were not printed correctly when they exceeded 14 characters. Therefore, I fixed this problem with this commit.

Both SSID and PASSWD are 13 characters (1234567890abc).

SSID is 13 characters (1234567890abc), and PASSWD is 14 characters (1234567890abcd).

Both SSID and PASSWD are 14 characters (1234567890abcd).


After the fix:
Both SSID and PASSWD were printed correctly even they are 14 characters (1234567890abcd).

SSID and PASSWD would be printed correctly if both are under 13 characters, but they wouldn’t be printed correctly if both exceed 14 characters. So I fixed this problem with this commit.
@sssscccc210 sssscccc210 changed the title Fixed an issue where SSID and PASSWD were not printed correctly if they exceeded 14 characters. [WiFiSetting] Fixed an issue where SSID and PASSWD were not printed correctly if they exceeded 14 characters. Sep 19, 2024
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.

1 participant