slither icon indicating copy to clipboard operation
slither copied to clipboard

Add reference urls to json output?

Open ghbren opened this issue 3 years ago • 2 comments

Describe the desired feature

Slither only outputs reference urls in the txt output. Would it be possible to add the urls to the json output too?

Thanks

ghbren avatar Jun 04 '22 03:06 ghbren

Hi I am interested in adding this feature.

Do we want a Reference URL in each object in the detector array? In the stdout output, functions with the same detectors are grouped together and the Reference URL is dumped after each block. but there cant be such blocks in the json. Meaning the reference url for "unchecked low-level" will appear for each unchecked low-level instance within a contract. Ex- for tests/test_function.sol, the reference_url will be present in 5 objects with check: "unchecked-lowlevel" instead of being shown only once

aranyadan avatar Aug 24 '22 18:08 aranyadan

opened a PR in #1360 assuming its fine to have a reference dumped for each detector object

aranyadan avatar Aug 25 '22 01:08 aranyadan