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

Error when importing text files or PDFs #6942

Closed
yassine-ouaamou opened this issue May 13, 2024 · 3 comments · Fixed by #6974
Closed

Error when importing text files or PDFs #6942

yassine-ouaamou opened this issue May 13, 2024 · 3 comments · Fixed by #6974
Assignees
Labels
bug use for describing something not working as expected critical use to identify critical bug to fix ASAP solved use to identify issue that has been solved (must be linked to the solving PR)
Milestone

Comments

@yassine-ouaamou
Copy link
Member

yassine-ouaamou commented May 13, 2024

Description

When importing a PDF/TXT file, an error occurs in some cases

Environment

Testing

Reproducible Steps

Steps to create the smallest reproducible scenario:

  1. Create a report
  2. Upload a PDF in the data section (example: https://www.cisa.gov/sites/default/files/2024-05/aa24-131a-joint-csa-stopransomware-black-basta_1.pdf)

Expected Output

The file should be uploaded then imported

Actual Output

Error when importing: list index out of range

Additional information

Screenshots (optional)

image

@yassine-ouaamou yassine-ouaamou added bug use for describing something not working as expected needs triage use to identify issue needing triage from Filigran Product team labels May 13, 2024
@yassine-ouaamou
Copy link
Member Author

The same issue happens with txt files also. I can create a new issue if needed.
Here is an example of a txt file:
txt_file.txt

@jborozco jborozco added critical use to identify critical bug to fix ASAP and removed needs triage use to identify issue needing triage from Filigran Product team labels May 13, 2024
@jborozco
Copy link
Member

The bug is moved to critical since it impacts majors clients

@jborozco
Copy link
Member

@SamuelHassine SamuelHassine added this to the Bugs backlog milestone May 13, 2024
@yassine-ouaamou yassine-ouaamou changed the title Error when importing some PDFs Error when importing text files or PDFs May 13, 2024
@Kedae Kedae self-assigned this May 14, 2024
@richard-julien richard-julien self-assigned this May 14, 2024
richard-julien added a commit that referenced this issue May 14, 2024
@richard-julien richard-julien linked a pull request May 14, 2024 that will close this issue
richard-julien added a commit that referenced this issue May 14, 2024
@richard-julien richard-julien added the solved use to identify issue that has been solved (must be linked to the solving PR) label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug use for describing something not working as expected critical use to identify critical bug to fix ASAP solved use to identify issue that has been solved (must be linked to the solving PR)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants