Ecdar / Ecdar/j-Ecdar

Logging printStackTrace and err output

Open
#91 0 comments 0 reactions 0 assignees View on GitHub
bug enhancement
Dominant language
Java
Stars
4
Forks
9
PR merge metrics
No merged PRs in 30d

Description

In some cases ``System.err.println`` and ``printStackTrace`` is being used instead of a log function (See [InProcessServer](https://github.com/Ecdar/j-Ecdar/blob/d1d1feaad68fbe73026d35d2fc70cb770f5f5c53/test/connection/InProcessServer.java#L32), [GrpcServer](https://github.com/Ecdar/j-Ecdar/blob/9d53d7a104cab5067b7337fa6967e84e6d9d8e7a/src/connection/GrpcServer.java#L48) and probably more).
* The ``printStackTrace`` should have a logging function which I believe should have the same urgency for all exceptions.
* The ``System.err.println`` should maybe be a fatal log.

Also we have to consider when we should use ``System.err.println```for ouput instead of ``stdio``.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by inspecting test/connection/InProcessServer.java at line 32 and src/connection/GrpcServer.java at line 48, then search for other uses of System.err.println and printStackTrace. Review the project's existing log functions and decide how exception urgency and stderr versus standard output should be handled; done means the direct output uses the agreed logging approach consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
observability-sre
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.