microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform
MIT License
2.82k stars 198 forks source link

Make `debug logs get` behave more intuitively #3573

Closed kananb closed 1 year ago

kananb commented 1 year ago

Summary of the Pull Request

A few facets to this PR:

  1. Add an optional --out_dir parameter to specify where to download the logs.
  2. Deprecate the --last and --all options, making them no-ops.
  3. Update the behavior to now always download all available logs (see #3572).
  4. Update the logs to the console to reflect the new behavior.

PR Checklist

Info on Pull Request

What does this include?

Validation Steps Performed

Local validation

codecov-commenter commented 1 year ago

Codecov Report

Merging #3573 (d4769e6) into main (09c8579) will decrease coverage by 0.01%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3573      +/-   ##
==========================================
- Coverage   39.01%   39.00%   -0.01%     
==========================================
  Files         274      302      +28     
  Lines       32396    36963    +4567     
==========================================
+ Hits        12639    14419    +1780     
- Misses      19757    22544    +2787     

see 28 files with indirect coverage changes