KasperskyLab/TinyCheck

Frontend / admin: One SSID breaks config.yaml formatting for "network:"

posixpoet opened this issue · 2 comments

When removing all SSIDs unter admin page "MANAGE DEVICE/Network config" and leaving only one, the config.yaml modified.
The modification is subesquently parsed errornously.

Previous setting under "network:/ssids:" (config.yaml):

 ...
 network:
  in: wlan1
  internet_check: http://example.com
  out: wlan0
  ssids:
  - skynet
  - wireless
  tokenized_ssids: true
 ...

After removal of all and addtion of SSID "TEST":

 ...
 network:
  in: wlan1
  internet_check: http://example.com
  out: wlan0
  ssids: TEST
  tokenized_ssids: true
 ...

"TEST" is subsequently then rendered as SSIDs "T", "E", "S", "T".
Expected behaviour is restored when manualyl editing config.yaml to

 ...
  ssids: 
  - TEST
 ...

Thanks for the reporting, I've done a commit which is correcting this issue and others related to the configuration API.
86193d9

Do not hesitate to close the issue if its OK for you.
Thanks for your feedback!
Félix.

Well done!
(Sorry, didn't offer the patch myself. Will do better.)