Skip to content

Conversation

@Jnoack331
Copy link
Contributor

This adds extra information to the detail page and would help if sending via the files somehow failes.

@shyim
Copy link
Member

shyim commented Mar 10, 2025

When you need it only on the detail page, we should extract it from EML. or what do you think @tinect

@Jnoack331
Copy link
Contributor Author

@shyim @tinect The idea was also to have a database backup of the CC/BCC when somehow the EML files have wrong permissions or some other files problem.

@tinect
Copy link
Member

tinect commented Mar 12, 2025

I agree with @shyim.
With the introduction of the EML-File, we plan to remove database entries reflecting the email content to save storage and mysql-traffic.

Brief overview of the plan: HTML will be stripped and TO and subject might stay. for searchability. Detailed information will then be loaded from the eml file if requested on the detail page.

Emails are stored securely in the private filesystem. Filesystem access issues are outside the scope of plugins and should be addressed separately.

Thank you anyway, @Jnoack331 🤗

@tinect
Copy link
Member

tinect commented Mar 15, 2025

Hey @Jnoack331 would you like to start implementing loading the CC and BCC from the file?

@Jnoack331
Copy link
Contributor Author

@tinect sure, i can take a look 👍

@Jnoack331
Copy link
Contributor Author

@tinect @shyim i added the changes to read the CC and BCC from the eml file. Is this OK or do you want any changes?

@Jnoack331 Jnoack331 force-pushed the feat/add-support-for-database-cc-and-bcc branch from 3b58fa0 to bb7ba4e Compare April 8, 2025 13:56
@Jnoack331
Copy link
Contributor Author

Is it fine like this, or do you need any changes?

@Jnoack331 Jnoack331 changed the title feat: add support for database CC and BCC feat: show EML CC and BCC in mail detail page Aug 11, 2025
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 this pull request may close these issues.

3 participants