Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adjusted WARC-Date for Revisit-Records #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

awaldlza
Copy link

Fix for Issue 31 (hopefully):

Changed method:
make_index_revisit

Information about the WARC-Date of the index record is passed on to the create_revisit_record-method, so that the revisit record gets the same WARC-Date entry as the index record.
Accordingly, the revisit record also gets a WARC-Creation-Date with the actual timestamp of the creation of the record.

As far as tested it works both with --fixed-dt and with the original time stamp of files that are "warced".

Adapted method make_index_revisit: When creating the revisit record the information of the WARC Date of the index/default file is passed on so that the revisit record has the same WARC-Date as the index file. This is helpful in Playback. The actual creation date is added is WARC-Creation-Date.
corrected creation_date
@Shrinks99 Shrinks99 self-requested a review October 17, 2024 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

1 participant