eclipse-ee4j / eclipse-ee4j/krazo
Add logging to PropertyUtils
- 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
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