[GH-ISSUE #82] password issue #60

Closed
opened 2026-02-28 01:23:10 +03:00 by kerem · 17 comments
Owner

Originally created by @ratzl on GitHub (Jan 26, 2016).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/82

When I try to enter a password in the configuration page, it's not working when the
password contains the + sign

example password: +Test- (not working)
example password: -Test- ( working)

Originally created by @ratzl on GitHub (Jan 26, 2016). Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/82 When I try to enter a password in the configuration page, it's not working when the password contains the + sign example password: +Test- (not working) example password: -Test- ( working)
kerem closed this issue 2026-02-28 01:23:10 +03:00
Author
Owner

@shirish47 commented on GitHub (Jan 27, 2016):

well I am not sure whats the problem with me when I try to connect after configuration it fails even though password is correct... it seems that there is some extra character.. or something .. my password contains alphabets and a number and "(" ")"..... previous libraries used to work...

<!-- gh-comment-id:175615300 --> @shirish47 commented on GitHub (Jan 27, 2016): well I am not sure whats the problem with me when I try to connect after configuration it fails even though password is correct... it seems that there is some extra character.. or something .. my password contains alphabets and a number and "(" ")"..... previous libraries used to work...
Author
Owner

@tzapu commented on GitHub (Jan 27, 2016):

thanks for the reports,
@ratzl & @shirish47 could you please let me know how your password ends up in the url string after save? it should be in the p= var

<!-- gh-comment-id:175622387 --> @tzapu commented on GitHub (Jan 27, 2016): thanks for the reports, @ratzl & @shirish47 could you please let me know how your password ends up in the url string after save? it should be in the p= var
Author
Owner

@shirish47 commented on GitHub (Jan 27, 2016):

yes it comes that way ..accept "(" turn to %28 and ")" to %29 I think that is normal even when wifi trys to connect it shows that password correctly in the debug msgs.

<!-- gh-comment-id:175627607 --> @shirish47 commented on GitHub (Jan 27, 2016): yes it comes that way ..accept "(" turn to %28 and ")" to %29 I think that is normal even when wifi trys to connect it shows that password correctly in the debug msgs.
Author
Owner

@shirish47 commented on GitHub (Jan 27, 2016):

when I reset it I get this ..
WM: AutoConnect
*WM: Reading SSID
*WM: SSID:
*WM: UNICORN
*WM: Reading Password
*WM: Password:
************ // it shows correct password but I removed it here
*WM: Connecting as wifi client...
*WM: Connection result:
*WM: 1
*WM:
*WM: Configuring access point...
*WM: AutoConnectAP
*WM: AP IP address:
*WM: 192.168.4.1
*WM: HTTP server started

<!-- gh-comment-id:175629757 --> @shirish47 commented on GitHub (Jan 27, 2016): when I reset it I get this .. _WM: AutoConnect *WM: Reading SSID *WM: SSID: *WM: UNICORN *WM: Reading Password *WM: Password:_***********\* // it shows correct password but I removed it here *WM: Connecting as wifi client... *WM: Connection result: *WM: 1 *WM: *WM: Configuring access point... *WM: AutoConnectAP *WM: AP IP address: *WM: 192.168.4.1 *WM: HTTP server started
Author
Owner

@tzapu commented on GitHub (Jan 27, 2016):

are you using 0.6 release ? or latest from github
what esp8266 core version are you using?

<!-- gh-comment-id:175633813 --> @tzapu commented on GitHub (Jan 27, 2016): are you using 0.6 release ? or latest from github what esp8266 core version are you using?
Author
Owner

@shirish47 commented on GitHub (Jan 27, 2016):

version 2 for esp8266 core and for this library its 0.6

<!-- gh-comment-id:175642850 --> @shirish47 commented on GitHub (Jan 27, 2016): version 2 for esp8266 core and for this library its 0.6
Author
Owner

@ratzl commented on GitHub (Jan 28, 2016):

*WM:
*WM: AutoConnect
*WM: Reading SSID
*WM: SSID:
*WM: rapi-wlan
*WM: Reading Password
*WM: Password: -12Li5a3- //password was -12Li5a3-
*WM: Connecting as wifi client...
*WM: Connection result:
*WM: 3
*WM: IP Address:
*WM: 134.103.68.138

*WM:
*WM: AutoConnect
*WM: Reading SSID
*WM: SSID:
*WM: rapi-wlan
*WM: Reading Password
*WM: Password: 12Li5a3- //password was +12Li5a3-
*WM: Connecting as wifi client...
*WM: Connection result:
*WM: 4
*WM:
*WM: Configuring access point...
*WM: ESP125778
*WM: AP IP address:
*WM: 192.168.4.1
*WM: HTTP server started

<!-- gh-comment-id:176157531 --> @ratzl commented on GitHub (Jan 28, 2016): *WM: *WM: AutoConnect *WM: Reading SSID *WM: SSID: *WM: rapi-wlan *WM: Reading Password *WM: Password: -12Li5a3- //password was -12Li5a3- *WM: Connecting as wifi client... *WM: Connection result: *WM: 3 *WM: IP Address: *WM: 134.103.68.138 *WM: *WM: AutoConnect *WM: Reading SSID *WM: SSID: *WM: rapi-wlan *WM: Reading Password *WM: Password: 12Li5a3- //password was +12Li5a3- *WM: Connecting as wifi client... *WM: Connection result: *WM: 4 *WM: *WM: Configuring access point... *WM: ESP125778 *WM: AP IP address: *WM: 192.168.4.1 *WM: HTTP server started
Author
Owner

@domonetic commented on GitHub (Jan 28, 2016):

What value returns getPassword()?
there may be a bug in the password storage from portal

<!-- gh-comment-id:176163409 --> @domonetic commented on GitHub (Jan 28, 2016): What value returns getPassword()? there may be a bug in the password storage from portal
Author
Owner

@domonetic commented on GitHub (Jan 28, 2016):

here the error:
in urldecore if char == '+'

if (*src == '+') {
decoded += ' ';
*src++;
} else {

<!-- gh-comment-id:176168792 --> @domonetic commented on GitHub (Jan 28, 2016): here the error: in urldecore if char == '+' if (*src == '+') { decoded += ' '; *src++; } else {
Author
Owner

@shirish47 commented on GitHub (Jan 29, 2016):

ok can you explain the error? is it encoding it before saving to file and then decoding it and having trouble. but my password does not include any other characters other than "(" and ")"...

<!-- gh-comment-id:176633696 --> @shirish47 commented on GitHub (Jan 29, 2016): ok can you explain the error? is it encoding it before saving to file and then decoding it and having trouble. but my password does not include any other characters other than "(" and ")"...
Author
Owner

@shirish47 commented on GitHub (Jan 29, 2016):

this library's old (0.5) version works fine with my code.

<!-- gh-comment-id:176636066 --> @shirish47 commented on GitHub (Jan 29, 2016): this library's old (0.5) version works fine with my code.
Author
Owner

@tzapu commented on GitHub (Jan 29, 2016):

guys, sorry about this, i m away until middle of next week and can only look/do at minor stuff, will try looking more next week

<!-- gh-comment-id:176659995 --> @tzapu commented on GitHub (Jan 29, 2016): guys, sorry about this, i m away until middle of next week and can only look/do at minor stuff, will try looking more next week
Author
Owner

@shirish47 commented on GitHub (Jan 29, 2016):

no problem buddy .. thanks :)

<!-- gh-comment-id:176732491 --> @shirish47 commented on GitHub (Jan 29, 2016): no problem buddy .. thanks :)
Author
Owner

@domonetic commented on GitHub (Jan 29, 2016):

@shirish47
I really do not understand that urldecode , but I think that's the problem with special characters.

<!-- gh-comment-id:176742010 --> @domonetic commented on GitHub (Jan 29, 2016): @shirish47 I really do not understand that urldecode , but I think that's the problem with special characters.
Author
Owner

@shirish47 commented on GitHub (Jan 29, 2016):

ya its I think something related to Base 64 I am not sure .. but I had similar issue with base 64 ..one of servers were not accepting my data after encoded to b64, because of null termination.. and its difficult to handle strings in C without null termination.

<!-- gh-comment-id:176743779 --> @shirish47 commented on GitHub (Jan 29, 2016): ya its I think something related to Base 64 I am not sure .. but I had similar issue with base 64 ..one of servers were not accepting my data after encoded to b64, because of null termination.. and its difficult to handle strings in C without null termination.
Author
Owner

@tzapu commented on GitHub (Feb 4, 2016):

i think i removed the + sign at some point for a specific purpose that i now can t remember :(
i think it may have been messing with spaces in ssids ... really can t remember
i think there might be a new version of url decoding coming soon as part of the core... i ll look into adding that

<!-- gh-comment-id:179784911 --> @tzapu commented on GitHub (Feb 4, 2016): i think i removed the + sign at some point for a specific purpose that i now can t remember :( i think it may have been messing with spaces in ssids ... really can t remember i think there might be a new version of url decoding coming soon as part of the core... i ll look into adding that
Author
Owner

@tzapu commented on GitHub (Feb 4, 2016):

hi all, this should be sorted in 0.9
i was double decoding as the server lib was already doing that
please test and reopen if needed

<!-- gh-comment-id:179819455 --> @tzapu commented on GitHub (Feb 4, 2016): hi all, this should be sorted in 0.9 i was double decoding as the server lib was already doing that please test and reopen if needed
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/WiFiManager#60
No description provided.