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.

Sign up for GitHub

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

Jump to bottom

Amends Vaulty data headers #231

Merged

abrignoni merged 1 commit into abrignoni:master from kibaffo33:master

Mar 10, 2022

Contributor

kibaffo33 commented Mar 10, 2022

Amends headers to match research rather than DB.

As queried here https://twitter.com/kibaffo33/status/1501212458417102858

@AlexisBrignoni ALEAPP etiquette question - when research shows a column header is inaccurate/better described another way (eg date modified is date added) - should the ALEAPP artefact reflect the DB or the research? #aleapp #dfir pic.twitter.com/Ird6vbousd
— Bob (@kibaffo33) March 8, 2022

I name the fields in a way that accurately reflects the proper interpretation of the data.

This is where having a blogpost for the research is key. You can put a link to it in the artifact report. See usage of the description variable below. pic.twitter.com/KQBRB2NSAV
— Brigs 💬 (@AlexisBrignoni) March 8, 2022

Thanks!


          Amends Vaulty data headers

13298ba

abrignoni merged commit da171f5 into abrignoni:master

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