Skip to content

Conversation

@ivanauth
Copy link
Contributor

Summary

This PR resolves three Maven dependency convergence errors reported in version 1.5.3:

  • jsr305: 3.0.1 vs 3.0.2
  • gson: 2.8.9 vs 2.11.0
  • protobuf-java: 3.25.8 vs 4.32.1

Changes

build.gradle:

  • Added jsr305 and findbugs annotations exclusions to protovalidate dependency
  • Added protobuf-java and gson exclusions to protobuf-java-util dependency
  • Added protobuf-java exclusion to proto-google-common-protos dependency
  • Added explicit jsr305:3.0.2 dependency

Testing with Maven Local

To test locally with Maven enforcer plugin:

  1. Build and publish to Maven local:
./gradlew clean publishToMavenLocal
  1. Create a test Maven project with the enforcer plugin:
<dependencies>
    <dependency>
        <groupId>com.authzed.api</groupId>
        <artifactId>authzed</artifactId>
        <version>0.0.0-SNAPSHOT</version>
    </dependency>
</dependencies>

<build>
    <plugins>
        <plugin>
            <groupId>org.apache.maven.plugins</groupId>
            <artifactId>maven-enforcer-plugin</artifactId>
            <version>3.6.2</version>
            <executions>
                <execution>
                    <id>enforce</id>
                    <configuration>
                        <rules>
                            <dependencyConvergence/>
                        </rules>
                    </configuration>
                    <goals>
                        <goal>enforce</goal>
                    </goals>
                </execution>
            </executions>
        </plugin>
    </plugins>
</build>
  1. Run Maven verify:
mvn clean verify

The three reported conflicts should be resolved. Note that three additional conflicts (error_prone_annotations, guava, j2objc-annotations) also appear but these existed before this PR and were not reported by users.

Related

Addresses remaining issues from #152

Add exclusions to resolve three dependency conflicts:
- jsr305: 3.0.1 vs 3.0.2
- gson: 2.8.9 vs 2.11.0
- protobuf-java: 3.25.8 vs 4.32.1
@ivanauth ivanauth requested a review from a team as a code owner October 15, 2025 21:49
Copy link
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense to me. Thank you!

@tstirrat15 tstirrat15 added this pull request to the merge queue Oct 15, 2025
Merged via the queue into authzed:main with commit 65e661a Oct 15, 2025
8 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants