jetty/jetty.project

Improve @ManagedAttribute processing

Open

#7561 opened on Feb 10, 2022

View on GitHub
 (2 comments) (0 reactions) (1 assignee)Java (3,701 stars) (1,913 forks)batch import
BugHelp Wanted

Description

Jetty version(s) 10.0.x

Description If @ManagedAttribute is put on a setter method, the processing of the annotation gets confused. See #7513 and comment https://github.com/eclipse/jetty.project/pull/7532#issuecomment-1029561508.

Contributor guide

Tech stack
java
Domain
backend
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
needs investigation
Prerequisites
JavaJetty internalsManagedAttribute API
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
35
Research direction
Investigate the behavior of @ManagedAttribute on setter methods. Review issue #7513 and the comment in PR #7532 (https://github.com/eclipse/jetty.project/pull/7532#issuecomment 1029561508) for context. Look at the annotation processing code in Jetty's source to understand how getter and setter annotations are handled. Propose a fix to correctly process setter annotations.