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

feat: add logger #82

Merged
merged 14 commits into from
Apr 6, 2024
Merged

feat: add logger #82

merged 14 commits into from
Apr 6, 2024

Conversation

Vinzent03
Copy link
Collaborator

close #70

@Vinzent03
Copy link
Collaborator Author

Wie findest du den Start mit der Benutzung von logger?
Alle Logs werden parallel in die Konsole und in eine Datei geschrieben.

@JanneckLange
Copy link
Collaborator

Wie findest du den Start mit der Benutzung von logger?

Generell nicht schlecht und können wir intern auch nutzen. Für #70 braucht es aber die anonymisierten Logs, die an uns gesendet werden können um Fehler auszuwerten. Das würde ich sicherheitshalber lieber trennen. Ein Methodenaufruf der klar macht, dass da keine personenbezogenen Daten drin geschrieben werden.

@Vinzent03
Copy link
Collaborator Author

Guter Einwand. Es sind jetzt zwei verschiedene Logger. Ein sensibler, der sowohl in die Konsole und in die Datei schreibt und einer nur für die Konsole. Außerdem werden die Logos für die Datei anders formatiert.

@Vinzent03 Vinzent03 marked this pull request as ready for review April 5, 2024 10:05
@Vinzent03
Copy link
Collaborator Author

Ich glaube nicht, dass es nötig ist Logs für ein Jahr zu speichern. Zum Überprüfen der Logs muss auch die gesamte Datei gelesen und geschrieben werden, was schneller geht, wenn es nicht unnötig viele sind. Habe es jetzt erstmal auf 30 Tage gesetzt.
Auf dem Loading screen habe ich jetzt erstmal einen einfachen Text zum Datenschutz geschrieben, sieht nicht gut aus, würde aber auf das redesign warten.

lib/screens/loading_info_screen.dart Show resolved Hide resolved
lib/utilities/logger.dart Outdated Show resolved Hide resolved
@JanneckLange
Copy link
Collaborator

JanneckLange commented Apr 5, 2024

Meine Konsolen Logs haben noch ein Formatierungsproblem. Sieht das bei dir anders aus?

Debugging-Konsole von VSC

Restarted application in 462ms.
flutter: \^[[38;5;12m13:48:47.965 (+0:00:00.122901)<…>
flutter: \^[[38;5;12min ResumeState<…>
flutter: \^[[38;5;12m13:48:48.107 (+0:00:00.264884)<…>
flutter: \^[[38;5;12mAppStateHandler: AppState.authenticated<…>
flutter: \^[[38;5;12m13:48:48.109 (+0:00:00.266099)<…>
flutter: \^[[38;5;12mScheduled sync data in 23:49:07.412975 in background<…>
flutter: \^[[38;5;12m13:48:48.109 (+0:00:00.266384)<…>
flutter: \^[[38;5;12mAppStateHandler: AppState.ready<…>

@Vinzent03
Copy link
Collaborator Author

Hmm bei mir funktionieren die Farben sehr gut, ich nutze es aber auch direkt im Terminal. Laut dem issue liegt es an iOS. Ich schaue mal, ob sich das beheben lässt.

@Vinzent03
Copy link
Collaborator Author

Ich habe den Logger jetzt nochmal bisschen geändert. In der Launch config von vscode habe ich Umgebungsvariablen hinzugefügt, mit denen die devtools logging Funktionen genutzt werden, die auch in der debug console in vscode angezeigt werden. Standardmäßig wird nach wie vor einfach in den stdout geprinted. Damit wird hoffentlich auch bei dir selbst mit iOS die Farben richtig angezeigt.

@JanneckLange
Copy link
Collaborator

Bildschirmfoto 2024-04-06 um 13 56 44 Logs sind besser

@Vinzent03 Vinzent03 merged commit 58ca9c2 into develop Apr 6, 2024
1 check passed
@Vinzent03 Vinzent03 deleted the feat/logger branch April 6, 2024 12:30
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.

2 participants