starred/WiFiManager

Fork 0

mirror of https://github.com/tzapu/WiFiManager.git synced 2026-04-27 17:15:53 +03:00

[GH-ISSUE #362] Content-length is not set properly #306

New issue

Closed

opened 2026-02-28 01:24:38 +03:00 by kerem · 8 comments

kerem commented

2026-02-28 01:24:38 +03:00

Owner

Originally created by @suculent on GitHub (May 3, 2017).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/362

Works on iPhone, but fails on certain Androids accessing the Captive Portal. Solution to fix this was straitforward... adding explicit Content-Length header before all ->send requests.

Originally created by @suculent on GitHub (May 3, 2017). Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/362 Works on iPhone, but fails on certain Androids accessing the Captive Portal. Solution to fix this was straitforward... adding explicit Content-Length header before all ->send requests.

kerem closed this issue

2026-02-28 01:24:39 +03:00

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@suculent commented on GitHub (May 3, 2017):

Following solution works for me:

github.com/suculent/WiFiManager@ada93b721a

@suculent commented on GitHub (May 3, 2017): Following solution works for me: https://github.com/suculent/WiFiManager/commit/ada93b721a235f4bae379e1804c32d962541b992

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@scop commented on GitHub (May 26, 2017):

I was bitten by this on Android 7.1.2, and the suggested change works for me too, please apply.

@scop commented on GitHub (May 26, 2017): I was bitten by this on Android 7.1.2, and the suggested change works for me too, please apply.

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@scop commented on GitHub (Jul 10, 2017):

@suculent, could you submit a pull request to get this fix in? Thanks!

@scop commented on GitHub (Jul 10, 2017): @suculent, could you submit a pull request to get this fix in? Thanks!

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@suculent commented on GitHub (Jul 10, 2017):

I've already done this quite a while ago https://github.com/tzapu/WiFiManager/pull/363

@suculent commented on GitHub (Jul 10, 2017): I've already done this quite a while ago https://github.com/tzapu/WiFiManager/pull/363

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@tzapu commented on GitHub (Jul 10, 2017):

thanks guys, merged

you need to download the latest master though, until there will be a release

@tzapu commented on GitHub (Jul 10, 2017): thanks guys, merged you need to download the latest master though, until there will be a release

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@tablatronix commented on GitHub (Sep 15, 2017):

The sdk seems to already send content-length headers, I wonder why android is so picky about the order maybe ?

@tablatronix commented on GitHub (Sep 15, 2017): The sdk seems to already send content-length headers, I wonder why android is so picky about the order maybe ?

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@suculent commented on GitHub (Sep 15, 2017):

It did not happen only on Android, at least I'm not the Android case, we use iPhones and iPad Pro to test WiFiManager in AP mode.

I need to find correct test case. After upgrading to latest WiFiManager I need to solve own issues first before going back to AP mode.

Odesláno z iPhonu

1. 2017 v 18:44, Shawn A notifications@github.com:

The sdk seems to already send content-length headers, I wonder why android is so picky about the order maybe ?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@suculent commented on GitHub (Sep 15, 2017): It did not happen only on Android, at least I'm not the Android case, we use iPhones and iPad Pro to test WiFiManager in AP mode. I need to find correct test case. After upgrading to latest WiFiManager I need to solve own issues first before going back to AP mode. Odesláno z iPhonu 15. 9. 2017 v 18:44, Shawn A <notifications@github.com>: > The sdk seems to already send content-length headers, I wonder why android is so picky about the order maybe ? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub, or mute the thread. >

kerem commented

2026-02-28 01:24:39 +03:00

Author

Owner

@tablatronix commented on GitHub (Sep 15, 2017):

Maybe this changed in SDK at some point, shrug

@tablatronix commented on GitHub (Sep 15, 2017): Maybe this changed in SDK at some point, shrug