Changes between Version 6 and Version 7 of Ticket #5146


Ignore:
Timestamp:
08/14/2018 03:40:27 AM (14 months ago)
Author:
hfiguiere
Comment:

With issue 6849 and 6850 we don't need to deal with checksum anymore.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5146 – Description

    v6 v7  
    1010There is a number of special cases here: 
    1111* Filter list header: if invalid, subscription status should be set to `synchronize_invalid_data` and further input ignored - no filter replacement. Otherwise the version number from the header should be saved in `requiredVersion` property. 
    12 * `! Checksum:` comment: not saved as filter, checksum validation should be performed (MD5 implementation required). If it fails, status should be set to `synchronize_checksum_mismatch` and the filters not replaced. 
     12* ~~`! Checksum:` comment: not saved as filter, checksum validation should be performed (MD5 implementation required). If it fails, status should be set to `synchronize_checksum_mismatch` and the filters not replaced.~~ obsolete with issue 6849. 
    1313* `! Redirect`, `! Homepage`, `! Title` comments: value should be saved in the corresponding subscription field. This field needs to be added for redirects, its value should not be serialized and should be used as an indicator for `Synchronizer` that it needs to handle a redirect. 
    1414* `! Expires` comment: the value should be returned by `finalize()` so that `Synchronizer` can update the expiration data properly. If `finalize()` returns 0 the default expiration interval is used.