mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-28 01:25:49 +03:00
[PR #467] [MERGED] Document that parameter id cannot contain spaces #1625
Labels
No labels
📶 WiFi
🕸️ HTTP
Branch
DEV Help Wanted
Discussion
Documentation
ESP32
Example
Good First Issue
Hotfix
In Progress
Incomplete
Needs Feeback
Priority
QA
Question
Task
Upstream/Dependancy
bug
duplicate
enhancement
invalid
pull-request
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/WiFiManager#1625
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/tzapu/WiFiManager/pull/467
Author: @lukepalmer
Created: 12/5/2017
Status: ✅ Merged
Merged: 1/31/2018
Merged by: @tablatronix
Base:
master← Head:master📝 Commits (1)
eb56062Document parameters📊 Changes
1 file changed (+4 additions, -0 deletions)
View changed files
📝
WiFiManager.h(+4 -0)📄 Description
It took me a while to figure out that custom parameters don't work if there is a space in the id parameter. It's not particularly useful to make this work, so just document that it doesn't.
The reason it doesn't work: the client will use the id parameter as the name of the field in the HTTP GET request that goes back to the server. Spaces and other special characters will get encoded, so "Blynk token" becomes "Blynk+token". The server doesn't do any kind of decoding so it gets confused and thinks the parameter just isn't there.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.