search

application-research / autoretrieve

A server to make GraphSync data accessible on IPFS
22 stars 7 forks source link

fix: return errors from WriteConfig in gen-config cli #98

Closed rvagg closed 2 years ago

rvagg commented 2 years ago

Discovered after trying to run gen-config post Docker usage which had my ~/.autoretrieve owned by root so unwritable but silently failing.

With this change I get:

2022-07-14T11:40:45.299+1000    FATAL   autoretrieve    autoretrieve/main.go:105        open /home/rvagg/.autoretrieve/config.yaml: permission denied
codecov-commenter commented 2 years ago

Codecov Report

Merging #98 (30a5ea2) into master (2d023c0) will increase coverage by 0.32%. The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master     #98      +/-   ##
=========================================
+ Coverage    8.04%   8.36%   +0.32%     
=========================================
  Files          12      12              
  Lines        1592    1590       -2     
=========================================
+ Hits          128     133       +5     
+ Misses       1460    1454       -6     
+ Partials        4       3       -1
Impacted Files Coverage Δ
main.go 0.00% <0.00%> (ø)
blocks/manager.go 85.00% <0.00%> (+5.00%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2d023c0...30a5ea2. Read the comment docs.