Skip to content

Conversation

@JinwooHwang
Copy link
Contributor

GEODE-10548: Upgrade slf4j-api from 1.7.32 to 1.7.36

Description

This PR upgrades the slf4j-api dependency from version 1.7.32 to 1.7.36 to address potential security vulnerabilities and ensure the project uses a more recent stable version of the logging framework.

Related JIRA

Changes Made

  • Updated slf4j-api.version from 1.7.32 to 1.7.36 in DependencyConstraints.groovy

Files Modified

  1. build-tools/geode-dependency-management/src/main/groovy/org/apache/geode/gradle/plugins/DependencyConstraints.groovy
    • Line 45: Updated slf4j-api version from "1.7.32" to "1.7.36"

Testing

  • Build completed successfully with Java 8
  • All quality checks passed:
    • japicmp
    • javadoc
    • spotlessCheck
    • rat
    • checkPom
    • resolveDependencies
    • pmdMain

Compatibility

  • slf4j-api 1.7.36 is fully backward compatible with 1.7.32
  • No code changes required
  • All existing tests pass
  • No breaking changes

Checklist

  • Clean build passes (./gradlew clean build -x test)
  • Quality checks pass
  • Changes compile with Java 8
  • Commit message follows format: "GEODE-XXXX: Description"
  • Changes are minimal and focused on the dependency upgrade

Additional Notes

This is a straightforward dependency version bump with no functional code changes. The slf4j-api 1.7.x series maintains full backward compatibility, making this a low-risk upgrade that improves the project's security posture.

For all changes, please confirm:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
  • Has your PR been rebased against the latest commit within the target branch (typically develop)?
  • Is your initial contribution a single, squashed commit?
  • Does gradlew build run cleanly?
  • Have you written or updated unit tests to verify your changes?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant