#3034 PR open: Use '2>>/dev/$SECRET_OUTPUT_DEV'

Labels: enhancement

jsmeix opened issue at 2023-08-02 09:09:

  • Type: Enhancement

  • Impact: Low

  • Reference to related issue (URL):

See
https://github.com/rear/rear/issues/2967
and
https://github.com/rear/rear/pull/3006
therein in particular
https://github.com/rear/rear/pull/3006#issuecomment-1590775360

  • How was this pull request tested?

See
https://github.com/rear/rear/pull/3006#issuecomment-1578565338

  • Brief description of the changes in this pull request:

Use

{ SECRET COMMAND ; } 2>>/dev/$SECRET_OUTPUT_DEV

instead of

{ SECRET COMMAND ; } 2>/dev/null

Reasoning:
See
https://github.com/rear/rear/issues/2967#issuecomment-1562732484
that reads (excerpt):

the

{ ... ; } 2>>/dev/$SECRET_OUTPUT_DEV

syntax makes it clear which redirections are
explicitly meant to hide secrets to distinguish them
from usual unwanted output discard via '2>/dev/null'

jsmeix commented at 2023-08-02 11:23:

This is not a critical issue
because before there was already 2>/dev/null
so nothing leaked into the log.

[Export of Github issue for rear/rear.]