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

[BUG] LOG_FILE_PATH env var doesn't work as expected #2894

Closed
0x009922 opened this issue Oct 20, 2022 · 1 comment
Closed

[BUG] LOG_FILE_PATH env var doesn't work as expected #2894

0x009922 opened this issue Oct 20, 2022 · 1 comment
Assignees
Labels
Bug Something isn't working iroha2-dev The re-implementation of a BFT hyperledger in RUST LTS Pre-alpha defect The defect was found at the operating of the pre-alpha. Had an impact on users. QA-confirmed This bug is reproduced and needs a fix

Comments

@0x009922
Copy link
Contributor

0x009922 commented Oct 20, 2022

GIT commit hash

b783f10

Minimum working example

Build Iroha and run it in terminal:

LOG_FILE_PATH=/dev/stdout ./iroha

Expected behaviour

Expected to see JSON logs in STDOUT.

Actual behaviour

I get an error:

Error: Failed to (de)serialize the field: log_file_path

Caused by:
   0: Name of the field: log_file_path
   1: expected value at line 1 column 1

Operating system

macOS 12.2.1

LibC type and version

Undefined

Current environment

Source code build

Who can help?

No response

Additional context

Known workaround is to escape the variable contents:

LOG_FILE_PATH="\"/dev/stderr\"" ./iroha

Also, it would nice if this variable is included into docker-compose.yml files located in the repo - thus it will be covered with tests, I guess.

@0x009922 0x009922 added Bug Something isn't working iroha2-dev The re-implementation of a BFT hyperledger in RUST LTS labels Oct 20, 2022
@AlexStroke AlexStroke added the Pre-alpha defect The defect was found at the operating of the pre-alpha. Had an impact on users. label Nov 7, 2022
@adaagava
Copy link

adaagava commented Jan 11, 2023

Actual on iroha2-dev
Works correctly with
"LOG_FILE_PATH": "/dev/stdout", in config file,
but failed from command prompt.

@AlexStroke AlexStroke added the QA-confirmed This bug is reproduced and needs a fix label Jan 11, 2023
@Erigara Erigara self-assigned this Jan 19, 2023
Erigara added a commit to Erigara/iroha that referenced this issue Jan 19, 2023
…alization

Signed-off-by: Shanin Roman <shanin1000@yandex.ru>
Erigara added a commit to Erigara/iroha that referenced this issue Jan 19, 2023
…alization

Signed-off-by: Shanin Roman <shanin1000@yandex.ru>
Erigara added a commit that referenced this issue Jan 19, 2023
Signed-off-by: Shanin Roman <shanin1000@yandex.ru>
@Erigara Erigara closed this as completed Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working iroha2-dev The re-implementation of a BFT hyperledger in RUST LTS Pre-alpha defect The defect was found at the operating of the pre-alpha. Had an impact on users. QA-confirmed This bug is reproduced and needs a fix
Projects
None yet
Development

No branches or pull requests

4 participants