eclipse-ee4j / eclipse-ee4j/krazo

Add logging to PropertyUtils

Open
#420 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
55
Forks
25
Avg merge
3h 25m
Merged PRs (30d)
3

Description

## Description
`PropertyUtils` is currently used when retrieving values from `MvcContext`. For debugging purposes, it would be helpful to log which property was read and what value was loaded.

Additionally, if a property conversion fails, the application currently crashes with an error. It would be better to log a `WARN` message and fallback to a default value instead of stopping the application.

## Open Questions / Discussion
* Logging all properties at once might be better suited for another location (e.g., an `Initializer`). If there is a more appropriate place for this, please let me know.
* Could this change become redundant due to integration with `Jakarta Config`? Please let me know if this issue is still relevant.

Contributor guide

Open the contributing guide

Research direction

Start by locating PropertyUtils and its use while retrieving values from MvcContext; inspect how property conversion failures are currently propagated. Confirm the intended logging location and default-value behavior with maintainers, then verify that successful reads log the property and loaded value and conversion failures emit WARN messages without crashing.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.