Skip to content

feat(securefile): Add --all flag to download all secure files

Description

  • feat(securefile): Add --all flag to download all secure files

This finishes migrating the functionality of the download-secure-files tool.

Related Issues

Resolves #7994 (closed)

How has this been tested?

  • Create a few secure files with different file extensions, including one with a directory in its path (eg: foo/bar.txt)
  • Download all files with --all successfully
  • Attempt to download a file with a fileID argument and --all, and receive (ERROR: all flag is not compatible with arguments)
  • Edit line 220 in download.go to be if checksum := "foo"; checksum != file.Checksum {
  • Recompile
  • Downloading secure files results in a checksum verification failed error
  • The file is not downloaded, and no tempfiles exist.
  • Try again with --no-verify and the files are successfully downloaded
  • Try again with --force-download and the files are successfully downloaded with a checksum verification failed message for each file
  • No tempfiles exist

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap
Edited by Fred Reinink

Merge request reports

Loading