-
Notifications
You must be signed in to change notification settings - Fork 218
-
Notifications
You must be signed in to change notification settings - Fork 218
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
Broken links in Whatsapp chats when attachments file names contain emojis #1836
Comments
I just reproduced the issue here and will try to fix it. |
I think I fixed the emoji related issue, but it seems there is another issue. @gfd2020, if you repeat the same test with a regular file name (like "books.pdf"), does it work? For reports (when items are exported as individual files using their hash as the file name) it works. However, when a folder is processed, it is not working here for audios, videos, documents and other file types, only for images. |
Hi @tc-wleite, I tested this yesterday with master after an user question and it is working for images, audios and videos when a mounted folder is processed. Not sure about 4.1.4, I'll check it. |
Please, test for a document (like a PDF) too. |
Probably I am doing something wrong here, as I tested using the version I just changed to fix the report issue. |
Yes, it works. |
Ok, thanks! |
I didn't find a WA sample with an attached PDF yet.
Makes sense! For audios and videos, there are a special handling. |
Actually I remember this issue with non image (documents) attachments from the past, but I didn't come up with a solution to open them from an alternative source if the first fails... |
It doesn't work. Sorry, I just tested the embedded player before. |
Thanks @lfcnassif! Later I will try to examine in more detail this non-related issue, and see if there is a reasonable solution. |
Yes, please! |
Do not replace emojis by images inside HTML tags (#1836)
I make more tests and did not work for documents, only images. |
Thanks again, @gfd2020! |
I'm confused now. When I generate the report with just a chat and I leave unchecked "Do not auto export chat attachments", images open and PDF does not. If I generate a report with a chat and individually tag an image and a pdf, both work. Shouldn't the "Do not auto export chat attachments" option export all media? |
It should. Are you testing attachment opening from the HTML report or from the IPED UI from the report?
Just for images. @tc-wleite just created #1841 fixing it for audios and videos. We are still not sure how to make it work for PDFs and other documents for these 2 situations at the same time: if you are opening them from a Chat into IPED HTML report or if you double clicked the Chat on UI after processing a mounted folder. |
Adding to @lfcnassif response...
Is the PDF file in the exported folder?
You mean the current version, right? |
I think it was a bug in my development environment.
Yes. Ok then. I just make more tests. I believe the behavior is as intended. Tested on IPED 4.1.4 Just check one whatsapp chat on IPED interface and make a report. Nothing more. Smartphone unzip ufdr ( image folder mounted to view report ): Checked "Do not auto export chat attachments" Smartphone unzip ufdr ( image folder unmonted to view report ): Checked "Do not auto export chat attachments" Smartphone ufdr file ( image folder mounted to view report ): Checked "Do not auto export chat attachments" Smartphone ufdr file ( image folder unmonted to view report ): Checked "Do not auto export chat attachments" |
I found a small bug in the emoji transformation. If the filename contains any emoji, the javascript link 'javascript:openIfExists' will be broken. Within IPED, the link will work, but not within reports. Example:
href="javascript:openIfExists('../../D/2/D2DED8529B0FBF1F60D700CE8B199999.pdf','file:///C:/test/files/books<img class="e1f4da">.pdf')"
Real path:
C:/test/files/books📚.pdf
PS: I don't know how whatsapp stores this filename internally, however it is possible to send files with emoji in the filename and they work normally.
This case that I tested and gave an error, it was a simulated whatsapp database.
The text was updated successfully, but these errors were encountered: