5
138
Fork
You've already forked goguma
18

WIP: Add a way to view logs #71

Draft
hhirtz wants to merge 1 commit from hhirtz/goguma:opentelemetry-support into master
pull from: hhirtz/goguma:opentelemetry-support
merge into: emersion:master
emersion:master
emersion:irc-socket
emersion:webcrypto-0.6.1
emersion:high-level-message
emersion:blocked
emersion:web
emersion:scrollable-positioned-list
emersion:l10n
emersion:ci-jvm-size
emersion:extended-isupport
emersion:drop-flutter-background
emersion:dup-main
emersion:android-decl-plugins
Contributor
Copy link

this adds a link to the logs in the settings

for now this is the bare functionality to show logs.

TODO

  • export to file
  • (削除) search through logs (削除ここまで) --> let's just export to file and search there
  • add a scrollbar to easily go through records when there are many
  • add a button to refresh the logs in the log page
    • ... or do we actually want logs to be shown as they come in realtime?
  • log irc traffic in release builds
    • redact AUTHENTICATE payloads
    • redact webpush keys?? idk
    • redact other stuff....

i don't think we specifically want logs to persist between restarts. at least for chathistory problems showing current logs should be enough

this adds a link to the logs in the settings for now this is the bare functionality to show logs. # TODO - [ ] export to file - [x] ~search through logs~ --> let's just export to file and search there - [ ] add a scrollbar to easily go through records when there are many - [ ] add a button to refresh the logs in the log page - [ ] ... or do we actually want logs to be shown as they come in realtime? - [ ] log irc traffic in release builds - [ ] redact AUTHENTICATE payloads - [ ] redact webpush keys?? idk - [ ] redact other stuff.... i don't think we specifically want logs to persist between restarts. at least for chathistory problems showing current logs should be enough
Add a way to view logs
All checks were successful
builds.sr.ht/android Job completed
e0636233fc 
this adds a link to the logs in the settings
for now this is the bare functionality to show logs.
All checks were successful
builds.sr.ht/android Job completed
This pull request has changes conflicting with the target branch.
  • lib/logging.dart
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u opentelemetry-support:hhirtz-opentelemetry-support
git switch hhirtz-opentelemetry-support

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff hhirtz-opentelemetry-support
git switch hhirtz-opentelemetry-support
git rebase master
git switch master
git merge --ff-only hhirtz-opentelemetry-support
git switch hhirtz-opentelemetry-support
git rebase master
git switch master
git merge --no-ff hhirtz-opentelemetry-support
git switch master
git merge --squash hhirtz-opentelemetry-support
git switch master
git merge --ff-only hhirtz-opentelemetry-support
git switch master
git merge hhirtz-opentelemetry-support
git push origin master
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
emersion/goguma!71
Reference in a new issue
emersion/goguma
No description provided.
Delete branch "hhirtz/goguma:opentelemetry-support"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?