JeremySun0731 commented on PR #1558: URL: https://github.com/apache/commons-lang/pull/1558#issuecomment-3738429717
While reviewing PR #1558 with LANG-1711 in mind, I noticed that not all reflective access paths appear to be centralized in AbstractReflection yet. In particular, ReflectionDiffBuilder still uses FieldUtils.readField(field, target, true), which forces accessibility unconditionally and bypasses the accessibleFlag mechanism introduced here. This means the new opt-in accessibility strategy does not currently apply to ReflectionDiffBuilder in the same way as ReflectionToStringBuilder and others. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
