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

BUG: Email tracking issue #3123

Closed
5 of 6 tasks
Rosencrantz opened this issue Jun 6, 2023 · 4 comments
Closed
5 of 6 tasks

BUG: Email tracking issue #3123

Rosencrantz opened this issue Jun 6, 2023 · 4 comments
Assignees
Labels
bug Things that should work, but don’t Critical Issue that requires prompt attention data-desk

Comments

@Rosencrantz
Copy link
Contributor

Rosencrantz commented Jun 6, 2023

Describe the bug
Aleph does not always show the full contents of an email requiring the user to download and open the email in another tool. Additionally, for some emails, attachments are not displayed.

Examples:

This is due to multiple related problems in ingest-file, followthemoney, and Aleph. I‘ve broken them down into separate issues. I’d recommend tackling them roughly in the following order:

Please note that due to the fact part of the root cause for this bug is in ingest-file, we won’t be able to resolve it completely for existing entities without re-ingesting them.

Implementing #3162 will allow users to at least view all contents (even if an email was ingested before the fix is rolled out). However, users will need to be aware that they might need to switch between the HTML and plaintext view to see all contents. We should consider it has a quick fix/temporary workaround while we work on #3161.

Once #3161 is fixed, users should be able to see complete email previews even without switching between HTML and plaintext view. However, the fix will not apply to emails that have been ingested before it has been rolled out.

Regarding #3165, it’s likely that the fix will also not apply to emails that have been ingested before it has been rolled out. However, we don’t know yet what exactly is the root cause here, so it’s a bit too early to give a definitive answer.

@Rosencrantz Rosencrantz added bug Things that should work, but don’t triage These issues need to be reviewed by the Aleph team data-desk Critical Issue that requires prompt attention labels Jun 6, 2023
@Rosencrantz Rosencrantz assigned Rosencrantz and stchris and unassigned Rosencrantz Jun 6, 2023
@Rosencrantz Rosencrantz removed the triage These issues need to be reviewed by the Aleph team label Jun 6, 2023
@jlstro
Copy link
Contributor

jlstro commented Jun 9, 2023

Hi,

I did more digging and made a very strange observation. I found at least one case where Aleph was showing part of the email, but the rest seems to be indexed - it shows up in a search when I search for a keyword that's in the non-visible part. (I can't share the details on what to search for and where here, it's an ongoing investigation).

In that case, the email contains a forwarded email that's attached inline. Then there's an email body above the inline attachment. Aleph only shows the forwarded email, not the original email body (but it appears to be searchable).

The original email also has 4 attachments, Aleph shows none.

@jlstro
Copy link
Contributor

jlstro commented Jun 13, 2023

Here is another one that's displayed incorrectly / does not have attachments.

@tillprochaska
Copy link
Contributor

tillprochaska commented Jun 20, 2023

@jlstro @brrttwrks I have updated the issue description with more details based on what I’ve found debugging the issue. This is actually a bigger issue with multiple separate causes. However, we will likely be able to implement at least a temporary workaround in a timely manner.

The technical details are unfortunately a bit more complex. If you’re interested in the details, feel free to look at the descriptions of the individual issues mentioned in the top-level comment or the Google Doc I shared with you. If something is unclear or confusing, feel free to reach out!

@tillprochaska
Copy link
Contributor

Going to close this as all but one issues have fixes that will be included in the next Aleph release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Things that should work, but don’t Critical Issue that requires prompt attention data-desk
Projects
None yet
Development

No branches or pull requests

4 participants