COVESA / dlt-daemon

Diagnostic Log and Trace.
https://covesa.github.io/dlt-daemon/
Mozilla Public License 2.0
381 stars 295 forks source link

dlt-common: add new config to remove date and time in storage header #632

Open ahmetfindi opened 6 months ago

ahmetfindi commented 6 months ago

Add a new OfflineTraceUseUptimeOnly toggle in dlt.conf With this toggle you can supress real date and time from the dlt message storage header and from offline trace filenames This toogle is for data protection purposes.

The program was tested solely for our own use cases, which might differ from yours. Licensed under Mozilla Public License Version 2.0

Ahmet Findikci ahmet.findikci@mercedes-benz.com, Mercedes-Benz Tech Innovation GmbH, imprint

michael-methner commented 5 months ago

Hello @ahmetfindi , thanks for your contribution. Can you remove the commit "[dlt-qnx-system: prevent unnecessary logging when buffer space is full]" from this PR as it is unrelated to the new config for date and time.

After discussion with the team, I have created the PR #635 which tries to fix dlt-qnx-system. Let's move the discussion the new PR.

ahmetfindi commented 5 months ago

Hello @michael-methner I've taken out the commit, I'll look at your PR.

michael-methner commented 5 months ago

Hello @ahmetfindi , thanks for the rework. From code review, I am fine with the change. @minminlittleshrimp can you verify the feature before merge? Thanks.

ahmetfindi commented 4 months ago

Hi @michael-methner @minminlittleshrimp how do we proceed with this MR, can we merge it?