Skip to content
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

Not able to see complete data #33

Closed
jayeshagwan1 opened this issue Dec 18, 2019 · 3 comments · Fixed by #35
Closed

Not able to see complete data #33

jayeshagwan1 opened this issue Dec 18, 2019 · 3 comments · Fixed by #35
Assignees

Comments

@jayeshagwan1
Copy link

jayeshagwan1 commented Dec 18, 2019

I followed the installation steps. Tried to run on files and db both, but getting the same output
image

Can anyone help on what I am missing ?

@vrajat vrajat self-assigned this Dec 18, 2019
@vrajat
Copy link
Member

vrajat commented Dec 18, 2019

I've found a bug. The application does not handle a single file correctly. I've been testing with a directory of files. I'll release a new version with the fix asap.

piicatcher files --path tests/samples/ --output-format json
[
  {
    "Mime/Type": "text/plain",
    "path": "tests/samples/sample-data.csv",
    "pii": [
      {
        "__enum__": "PiiTypes.EMAIL"
      },
      {
        "__enum__": "PiiTypes.CREDIT_CARD"
      },
      {
        "__enum__": "PiiTypes.PERSON"
      },
      {
        "__enum__": "PiiTypes.LOCATION"
      },
      {
        "__enum__": "PiiTypes.BIRTH_DATE"
      }
    ]
  }
]

piicatcher files --path tests/samples/sample-data.csv --output-format json
[]

@vrajat
Copy link
Member

vrajat commented Dec 18, 2019

I've released a new version - 0.6.1. Please try it out and reopen the issue if it still doesnt work. Can you open a new issue for databases with details ? I'll take a look.

@jayeshagwan1
Copy link
Author

Thanks @vrajat. json format working fine.

image

Can we format ascii_table ?

image

I will open a new issue for databases with details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants