Skip to content

Commit

Permalink
and another bugfix for cached check sums reporting on severity of bug…
Browse files Browse the repository at this point in the history
… in UPGRADING
  • Loading branch information
petersilva committed Aug 9, 2021
1 parent a694093 commit a03386d
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 1 deletion.
12 changes: 12 additions & 0 deletions doc/UPGRADING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,18 @@ Installation Instructions
git
---

2.21.08
-------


BUG FIXED by this release:

it looks like -n (checksum based on file name) was not working as intended... for a long time, as reported in
( https://github.com/MetPX/sarracenia/issues/377 ) when multiple file versions had different sizes, they
would not be identified as the same file for duplicate_suppression purposes, making the -n option quite
ineffective. Searching through the changelog, it looks like this has been present for a long time.


2.21.04
-------

Expand Down
6 changes: 5 additions & 1 deletion sarra/sr_post.py
Original file line number Diff line number Diff line change
Expand Up @@ -586,8 +586,12 @@ def compute_sumstr(self, path, fsiz):
self.logger.debug("mtime remembered by xattr")

attr_sum = xattr.get('sum')
if attr_sum[0] == self.sumflg :
if ( attr_sum[0] == self.sumflg ) :
return attr_sum

if hasattr(self.msg.headers,'sum') and ( attr_sum[0] == self.msg.headers['sum'][0] ) :
return attr_sum

else:
sumflg = self.sumflg

Expand Down

0 comments on commit a03386d

Please sign in to comment.