autopkg / nstrauss-recipes

Public AutoPkg recipes
23 stars 27 forks source link

VirusTotalReporter: Coerce summary results to string #71

Closed nstrauss closed 4 months ago

nstrauss commented 4 months ago

AutoPkg pretty much assumes everything will always be a string. Otherwise it gets angry. Coerce summary results to string, which can be modified as needed later in whatever post-processing is done. Previously results were mixed string and integer anyway, which didn't make sense.

The following items were queried from the VirusTotal database:
Traceback (most recent call last):
  File "/usr/local/bin/autopkg", line 2786, in <module>
    sys.exit(main(sys.argv))
  File "/usr/local/bin/autopkg", line 2782, in main
    return subcommands[verb]["function"](argv)
  File "/usr/local/bin/autopkg", line 2391, in run_recipes
    this_column = [len(row[column]) for row in rows]
  File "/usr/local/bin/autopkg", line 2391, in <listcomp>
    this_column = [len(row[column]) for row in rows]
TypeError: object of type 'int' has no len()

https://github.com/autopkg/autopkg/blob/8a12727820da6be40ee27530ac492c30283e54ca/Code/autopkg#L2388-L2392