Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(506)

Issue 348840043: [plaso] Migrated Safari cookies construct-based parser to use dtfabric #1893 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 weeks, 3 days ago by Joachim Metz
Modified:
2 days, 10 hours ago
Reviewers:
onager
CC:
kiddi, log2timeline-dev_googlegroups.com, romaing
Visibility:
Public.

Description

[plaso] Migrated Safari cookies construct-based parser to use dtfabric #1893

Patch Set 1 #

Patch Set 2 : Changes after merge and clean up #

Unified diffs Side-by-side diffs Delta from patch set Stats (+265 lines, -112 lines) Patch
M plaso/parsers/dtfabric_parser.py View 1 2 chunks +9 lines, -3 lines 0 comments Download
M plaso/parsers/safari_cookies.py View 1 6 chunks +135 lines, -109 lines 0 comments Download
A plaso/parsers/safari_cookies.yaml View 1 1 chunk +121 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Joachim Metz
3 weeks, 3 days ago (2018-05-27 11:16:58 UTC) #1
Joachim Metz
Code updated.
1 week, 3 days ago (2018-06-10 08:23:28 UTC) #2
onager
On 2018/06/10 08:23:28, Joachim Metz wrote: > Code updated. Reviewing on github
3 days, 15 hours ago (2018-06-17 12:29:23 UTC) #3
Joachim Metz
2 days, 10 hours ago (2018-06-18 17:50:45 UTC) #4
Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: review.py close dtfabric5
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b