Skip to content

Conversation

@ofostier
Copy link
Contributor

@ofostier ofostier commented Jun 6, 2024

Regarding the request 283.
I add the query parameters management.

@ofostier
Copy link
Contributor Author

ofostier commented Jun 6, 2024

Query parameters are :
ssid = string
password = string
encryptionMode = string WPA / WPA2-EAP / WEP
eapMethod = string default 'PWD'
eapIdentity = string
hidePassword = string true/false
hiddenSSID = string true/false
portrait = string true/false
additionalCards = number
hideTip = string true/false
lng = all available language id in translation.js file

@ofostier ofostier closed this Jun 6, 2024
@ofostier ofostier reopened this Jun 6, 2024
@millette
Copy link

Perhaps it would be best to use the #hash instead of query parameters since the #hash doesn't reach the server.

Related to #283 (proper issue id links the pr accrross github).

@bndw
Copy link
Owner

bndw commented Jun 12, 2024

@millette Good callout. I don't want this webapp to send private information to any server and query params would certainly do that. Using a hash instead is a clever workaround.

Further, I think the approach in this PR needs a little more thought. I don't love the implementation. The core logic should be factored out into a function and not jammed inline.

@ofostier
Copy link
Contributor Author

Hi,
Thanks to make me brainstorming on this
I reviewed the logic and implementation

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.

3 participants