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

Missing option to ommit the log files in mail sharing #47

Closed
Aiosa opened this issue Nov 28, 2022 · 1 comment
Closed

Missing option to ommit the log files in mail sharing #47

Aiosa opened this issue Nov 28, 2022 · 1 comment

Comments

@Aiosa
Copy link

Aiosa commented Nov 28, 2022

Adding such feature to the API should be a piece of cake, and it could be crucial for some people because:

  • they can reuse the module for content-related reports (i.e. wrong text there and there) as well where there is no need to include log files since such kind of mistakes could be just static content, e.g. in a database
  • various policies stress that users have options not to share their data, which is the case here: it is possible to manually delete these files from the mail draft, but it would be much more elegant if developers could let the user decide whether to share their data
  • showing the option 'do not include my data' could possibly make users hesitate less to share errors, although maybe without log files - further sharing could be communicated through individually
@Aiosa Aiosa changed the title Missing option to ommit the log files Missing option to ommit the log files in mail sharing Nov 28, 2022
@fdrault
Copy link
Member

fdrault commented Feb 17, 2023

Hi,
This lib goal is to write log into files with different rollings stategies and provides them.
You can freely access to all the files path with getLogFilePaths and pick what you want, according to your privacy policies.

sendLogFilesByEmail is designed for an easy access to logs, to debug or for beta testers. It is a convenient method for developers, but it is not designed to be a bug reporting tool in production

@Aiosa Aiosa closed this as completed Apr 4, 2023
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

No branches or pull requests

2 participants