Describe the bug
The plugin library allows for strings to act as booleans for plugin configuration. This is important, because with the current SHIELD CLI, an attempt to configure a boolean data member will instead configure a string (e.g. "false" instead of false`).
Plugin code here:
|
if s, ok := endpoint[key].(string); ok { |
The Web UI, upon seeing the string "false" in the edit target dialogue will show the checkbox as checked, or "true". This is likely because strings in Javascript are truthy, and we're likely doing a naive javascript truthiness check.
To Reproduce
Steps to reproduce the behavior:
- Set a target of type
fs's verbose property to false with the CLI.
- Go to the web UI
- edit that target
- see that the checkbox is checked
Expected behavior
I expected the checkbox to be representative of the behavior the plugin would have. In this case, I expected the checkbox to be off.
Screenshots
If applicable, add screenshots to help explain your problem.
SHIELD versions (please complete the following information):
- SHIELD Core: 8.7.1
- SHIELD CLI: 8.7.0
Browser version(s) (please complete if reporting a web UI bug):
- OS: Windows
- Browser: Firefox
- Version: 75.0
Additional context
Add any other context about the problem here.
Describe the bug
The plugin library allows for strings to act as booleans for plugin configuration. This is important, because with the current SHIELD CLI, an attempt to configure a boolean data member will instead configure a string (e.g.
"false" instead offalse`).Plugin code here:
shield/plugin/endpoint.go
Line 82 in be0688c
The Web UI, upon seeing the string "false" in the edit target dialogue will show the checkbox as checked, or "true". This is likely because strings in Javascript are truthy, and we're likely doing a naive javascript truthiness check.
To Reproduce
Steps to reproduce the behavior:
fs'sverboseproperty tofalsewith the CLI.Expected behavior
I expected the checkbox to be representative of the behavior the plugin would have. In this case, I expected the checkbox to be off.
Screenshots
If applicable, add screenshots to help explain your problem.
SHIELD versions (please complete the following information):
Browser version(s) (please complete if reporting a web UI bug):
Additional context
Add any other context about the problem here.