-
Notifications
You must be signed in to change notification settings - Fork 1
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
Read lnk file structure from file like object #18
Read lnk file structure from file like object #18
Conversation
In my opinion the |
Remove file-object test
I agree! Only some small changes have to be done in dissect.target and the lnk-file parser tool of dissect.shellitem. In this case it is only needed to change @Zawadidone could you change line 15 in https://github.com/fox-it/dissect.shellitem/blob/main/dissect/shellitem/tools/lnk.py to For dissect.target I created the following PR fox-it/dissect.target#224. Ill approve and run CI when the proposed changes are reflected! |
Codecov Report
@@ Coverage Diff @@
## main #18 +/- ##
==========================================
+ Coverage 69.07% 69.10% +0.02%
==========================================
Files 4 4
Lines 304 301 -3
==========================================
- Hits 210 208 -2
+ Misses 94 93 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Closes #17
After merge adjust changes in https://github.com/fox-it/dissect.target/blob/main/dissect/target/plugins/os/windows/lnk.py#L78 and add a test.