Skip to content
This repository has been archived by the owner on Aug 6, 2019. It is now read-only.

Implement Dump parsing (first pass) #10

Merged
merged 31 commits into from
Feb 6, 2019
Merged

Implement Dump parsing (first pass) #10

merged 31 commits into from
Feb 6, 2019

Conversation

rye
Copy link
Owner

@rye rye commented Feb 6, 2019

Closes #9.

rye added 26 commits February 6, 2019 11:39
@rye
main: Open the dump files
52a3c15 
For now, I'll use unwrap to make this die if the file cannot be opened
for reading.  In the future, it'll just want a match block around it
that errors out and exits.  Or maybe I should try the ? trait?

Food for thought.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Set up empty lib.rs file
a46e7d8 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
lib: Use external csv crate
1eb1de4 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
lib: Start on dump module
7212508 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
dump: Add five structs
b31a931 
To get started, we'll use these.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
dump: Add .read fn and use it in main
3128857 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
main: Resolve namespace errors by using nessify::*
a378860 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
cargo: Add serde and serde_derive as first-class dependencies
8d84feb 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
lib: Wire up serde and serde_derive
0f6e4fe 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Instantiate a csv::Reader and begin to use it
e448dd7 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Record: Add fields
33eb946 
These correspond one-to-one with the fields in a Nessus CSV file.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Record: derive Debug and Deserialize
e1c3873 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Record: Add serde mappings between field headers and fields
d0c284c 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
dump: Fill out fields in Plugin, Host, Detection, Dump structs
1b75e38 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Plugin: Add PartialEq trait to Plugin
1dfe104 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Plugin: Define Hash trait for Plugin as well
3f9f4ef 
This is done primarily because the cve field needs to be excluded.  A
plugin with a different cve field should not exist.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Plugin: Derive Clone, Eq traits
5f23ae8 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Host: Derive Clone, Hash, Eq, PartialEq traits
4557688 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Detection: Derive Clone, Hash, Eq, PartialEq traits
0ef7c7d 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Begin parsing the CSV file
68a100e 
N.b. that csv::Reader already uses BufReader---I initially planned on
adding it in here but plans changed.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Extract the Plugin fields from the Record
08a8077 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Also parse out the Host fields from the Record
8df72c8 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Fill Plugins into the HashSet
1886858 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Fill Hosts into their HashSet
434a21c 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Finish reading and parsing and such
3afb32a 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Dump::read: Add a couple of minutia
59d7c63 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye rye added a/main Of or relating to the executable-specific code a/lib Of or relating to the library in general t/new feature t/impl labels Feb 6, 2019
@rye rye self-assigned this Feb 6, 2019
rye added 5 commits February 6, 2019 15:31
@rye
dump: Move Record struct into its own file
db09533 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
dump: Move host into its own module
2c461d9 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
dump: Move Plugin struct into its own module
fcb89d9 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
dump: Move Detection struct into its own module
feb6c2d 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@rye
Begin testing dump module
4c3a1de 
Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>
@codecov-io
Copy link

Codecov Report

Merging #10 into master will increase coverage by 19.73%.
The diff coverage is 23.43%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #10       +/-   ##
===========================================
+ Coverage       0%   19.73%   +19.73%     
===========================================
  Files           1        4        +3     
  Lines          12       76       +64     
===========================================
+ Hits            0       15       +15     
- Misses         12       61       +49
Impacted Files Coverage Δ
src/main.rs 0% <0%> (ø) ⬆️
src/dump/plugin.rs 0% <0%> (ø)
tests/dump.rs 100% <100%> (ø)
src/dump.rs 21.95% <21.95%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b95381...4c3a1de. Read the comment docs.

@rye rye merged commit 2e5dbeb into master Feb 6, 2019
@rye rye deleted the dump branch April 15, 2019 00:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@rye rye

Labels
a/lib Of or relating to the library in general a/main Of or relating to the executable-specific code t/impl t/new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rye @codecov-io