Table format not displaying output properly for file system about piicatcher HOT 14 CLOSED

Now I remember why I had has_pii instead of pii column for databases. Piicatcher uses a library called tableprint to print the ascii table and it has its limitations. You've hit both of them.

1. It is hard to format long columns
2. It expects a terminal to print a table. It cant be redirected to a file.

I am not sure there is an easy fix for either of them. I suggest you use JSON to store to a file. In #51 I'll improve JSON format to also include the PII types.

from piicatcher.

Thanks. So can we save this data directly in db?

from piicatcher.

from piicatcher.

Thanks ! So we will have these options in the similar way as config file ?

from piicatcher.

Yes: https://tokern.io/docs/piicatcher/2-usage You can use --orm-host and other options. FYI: I am going to change those option names to "catalog-". For example, catalog-host etc. Right now, MySQL is supported
…
On Tue, Jan 21, 2020 at 12:10 PM jayeshagwan1 @.**> wrote: Thanks. So can we save this data directly in db? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#52?email_source=notifications&email_token=AAMP7GSCJI47OUAOBQMFVVDQ62KGXA5CNFSM4KJOA4P2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJOU3SI#issuecomment-576540105>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMP7GSCOAQRGJ2INZ7JSZLQ62KGXANCNFSM4KJOA4PQ .

Can you provide the exact command for this ? So I have 2 config files 1 for connecting db and read the db and other for inserting into db.

from piicatcher.

Single config file

 catalog_host=".."
 catalog_port="..."
 catalog_user="..."
 catalog_password="..."

[db]
...

Also can you give me feedback on the docs ?

https://tokern.io/docs/piicatcher/2-usage#configuration-file
https://tokern.io/docs/piicatcher/3-catalog#database

from piicatcher.

Docs look good, just one observation. In docs its hyphen eg catalog-host and in above response you mentioned catalog_host.
Also if we can add sample of command for each type, it would help.

from piicatcher.

from piicatcher.

Can we add one more column for Pii type ?

In the above screenshot, ACQUISITION_DATE if of which type of PII.

from piicatcher.

from piicatcher.

Aaah. Ok

from piicatcher.

@vrajat Can we keep output consistent for all ? In db, json file shows

If we try file type then response is:

So in file type we are not able to identify which column has PII data and what type of PII data. If file output would also show same as db one, it would be better to understand

from piicatcher.

from piicatcher.

I'll be removing support for files.

from piicatcher.