-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
printers should be extended (html format + all formats) #44
Comments
Also, this is kind of too far in the future, but if there are ever 3+ formats the --output-format flag may get enhanced in the sense that it accepts multiple flags, so that the user can cherrypick which output formats should be activated in the runtime. |
For implementing the HTML printer, the following points should receive special attention: |
For the table filtering, take a look at https://react-querybuilder.js.org/ and JsonLogic of it |
If possible, it would be convenient if the table rows were filled with the JSON data. This would reduce the amount of data saved, would allow for backwards compatibility (as only the new template + old JSON data is necessary), and would at it's core mean the user never needs to choose between the formats |
Currently, we only have the JSON printer. The current output format only logs responses. The structure does not log things such as startup options. We may want to clarify if that is an intended feature.
Additionally, bringing at least the HTML printer back would be nice. Coupled with some JS, it could enable dynamic filtering of results directly within that HTML output page for the user. That would be very convenient, as JSON is not suited for manual user processing.
Lastly, once there is the option to specify the format, there should be the possibility to signal to wenum to output in both HTML and JSON (practically output in all formats).
This has implications for the naming convention of the output file. With a single format, wenum should directly take the user specified file name as output. If more than one output format is specified, wenum should take the user file name as a base, and append each format in the form of '.format'. This seems to be a common convention on how to deal with one format vs. multiple formats specified (see nmap and ffuf)
The text was updated successfully, but these errors were encountered: