gnodet opened a new pull request, #24853:
URL: https://github.com/apache/camel/pull/24853

   ## Summary
   
   _Claude Code on behalf of Guillaume Nodet_
   
   Fix two genuine SonarCloud S1845 (identifier name differs only by case) 
issues:
   
   - **PubSubApiProcessor**: Make `LOG` field `static final` instead of 
non-static instance field. This eliminates the clash with the inherited `log` 
field from the superclass and follows the standard Camel logging convention.
   - **PGPDataFormatUtil**: Fix method name typo `findPrivateKeyWithkeyId` → 
`findPrivateKeyWithKeyId` (uppercase 'K'). The old method is deprecated for 
backwards compatibility and delegates to the correctly named one. Internal 
caller `DefaultPGPSecretKeyAccessor` updated to use the new name.
   
   ### SonarCloud context
   
   This PR is part of the BLOCKER remediation effort. Of the 46 S1845 issues, 
41 were triaged as won't-fix (intentional patterns: inherited logger shadowing, 
field vs constant naming, deprecated API migration, method overloads with 
different signatures). The remaining 5 included these 2 genuine fixes and 3 
test-infra `username`/`userName` deprecation transitions (marked as won't-fix 
since the naming is intentional during API migration).
   
   ## Test plan
   - [x] `mvn test -pl components/camel-crypto-pgp` — all 69 tests pass (22 
skipped)
   - [ ] CI full build
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to