Skip to content

Add jpeg exif parser#4941

Open
oh6hay wants to merge 10 commits into
log2timeline:mainfrom
oh6hay:add-jpeg-exif-parser
Open

Add jpeg exif parser#4941
oh6hay wants to merge 10 commits into
log2timeline:mainfrom
oh6hay:add-jpeg-exif-parser

Conversation

@oh6hay
Copy link
Copy Markdown

@oh6hay oh6hay commented Dec 12, 2024

One line description of pull request

Add a JPEG EXIF parser

Description:

Parses basic EXIF data from JPEG images. It seems introducing new dependencies isn't exactly encouraged due to possible license compatibility issues so I made a quick&dirty inline jpeg parser.

Notes:

All contributions to Plaso undergo code review.
This makes sure that the code has appropriate test coverage and conforms to the
Plaso style guide.

One of the maintainers will examine your code, and may request changes. Check off the items below in
order, and then a maintainer will review your code.

Checklist:

  • Automated checks (GitHub Actions, AppVeyor) pass
  • No new new dependencies are required or l2tdevtools has been updated
  • Reviewer assigned

@joachimmetz
Copy link
Copy Markdown
Member

Thanks for the proposed changes, please take a look at the style guide https://plaso.readthedocs.io/en/latest/sources/developer/Style-guide.html and please change the PR to align with that style

@oh6hay
Copy link
Copy Markdown
Author

oh6hay commented Dec 13, 2024

hi @joachimmetz , the style guide is rather concise, could you kindly point towards any discrepancies?

@joachimmetz
Copy link
Copy Markdown
Member

I'll take a closer look when time permits. In the meantime I recommend to have a look at similar parsers in Plaso. Unfortunately the style deviates from that at the moment, so merging the code might take a significant time (due to limited bandwidth on my end)

@oh6hay
Copy link
Copy Markdown
Author

oh6hay commented Dec 14, 2024

thanks @joachimmetz I'll have a look at other parsers. Are contributions to the style guide also welcome, to better define all aspects of the code and style conventions?

@joachimmetz
Copy link
Copy Markdown
Member

joachimmetz commented Dec 14, 2024

Are contributions to the style guide also welcome, to better define all aspects of the code and style conventions?

They are. But realize though that some of the style decision are historical and we prefer to try to keep the code mostly in the same style. So making changes such as adoptions of f-strings or type hints takes time.

@joachimmetz joachimmetz force-pushed the add-jpeg-exif-parser branch from f0a7d12 to 961afbf Compare May 11, 2026 05:15
@joachimmetz joachimmetz added this to the Parking/Temporary release milestone May 11, 2026
@joachimmetz joachimmetz self-assigned this May 11, 2026
Comment thread plaso/parsers/jpeg_exif.py Outdated
0xa408: 'Contrast',
0xa409: 'Saturation',
0xa40a: 'Sharpness',
0xa431: 'Body Serial Number',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Body Serial Number exists multiple times and will be overwriten in the annotation process

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants