mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #4275] Rework quiet, debug and normal console output #2603
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#2603
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
Originally created by @rusty-snake on GitHub (May 13, 2021).
Original GitHub issue: https://github.com/netblue30/firejail/issues/4275
We have multiple issues about how we print something to the console, which all go to the same code. Therefore I make this meta-issue.
#4168
Fixed in
65b7cd79cf#3125
Fixed in
65b7cd79cf#3503
#3354
#3749
#4328
#2322
There are some more which aren't handled here like:
Fixed in
9fd3560768and
9a84e4df60Good to read:
IMHO we should only print warnings and errors to stderr by default and be quiet for everything else. The current
quietwould be then a hard-queit if a program really need full ownership of stderr. In addition a--info/--verbose/--debug=1/--debug=little/whatever will show the current default output.All the warnings/errors should make clear that they are printed by firejail (e.g.
firejail:Error:).For the debug output it would be enough to print something like
@ghost commented on GitHub (May 13, 2021):
This reminds me of #3325, which never really got of the ground. @topimiettinen mentioned a possibility to improve errExit() and use it everywhere with a prefix. IMO that would solve this part of the issue. Just adding it here for future reference.
@slowpeek commented on GitHub (Jun 3, 2021):
I'm not sure is there a single reason to use stdout in firejail. It is a wrapper, hence it should not touch neither stdin nor stdout.