NICMx / FORT-validator

RPKI cache validator
MIT License
47 stars 23 forks source link

support ROA export in JSON format #47

Closed reschke closed 3 years ago

reschke commented 3 years ago
pcarana commented 3 years ago

Thanks for this PR! I'll adequate a couple of things in the next days.

One of them is to add an argument to set the output format, something like --output.format (allowed values: 'csv' or 'json'), and according to its value, print the output to whatever is set at --output.roa.

Regarding the TA information related to each ROA, let me see if I can do something efficient (I wouldn't like to hold a string on each record of the DB).

reschke commented 3 years ago

Yes.

As you can see, I chose the most simplistic approach. I appreciate if you can do it more elegantly.

reschke commented 3 years ago

@pcarana - thanks for reviewing and improving this!

pcarana commented 3 years ago

Great! Thanks for doing the merge, that was the next step. Sorry that this hasn't been moving faster, but right now I'm doing this on my free time.