-
Notifications
You must be signed in to change notification settings - Fork 700
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
Add UDF file support #941
Add UDF file support #941
Conversation
8ae41e1
to
86867d8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't tested yet, but from code review these are things I noticed.
003632c
to
071971f
Compare
b1b8370
to
4f265dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few more things.
5a28b15
to
523ab87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review looks good. Manual testing looks good (notes in Jira under CLAM-2322). I'm happy. Waiting to see it pass through existing Jenkins build/tests. As noted, we don't have a benign sample to test with, so no automated test at this time.
Add support for specifically for Beginning Extended Area Descriptor (BEA01) type of UDF files.
523ab87
to
2ca8903
Compare
rebased because jenkins macos build failed because of openssl3 changes |
Add support for the Beginning Area Descriptor (BEA01) UDF files.