Skip to content

Conversation

@thomasturrell
Copy link
Contributor

This pull request updates the Maven repository URLs in the pom.xml file to use the latest Sonatype endpoints for both snapshot and release deployments.

Repository configuration updates:

  • Updated the snapshotRepository URL to https://central.sonatype.com/repository/maven-snapshots/ to reflect the new Sonatype snapshot repository location.
  • Changed the repository URL to https://ossrh-staging-api.central.sonatype.com/service/local/ for deploying release artifacts to the new staging API endpoint.

@sonarqubecloud
Copy link

@thomasturrell thomasturrell merged commit 7cd3d74 into main Nov 14, 2025
7 checks passed
@thomasturrell thomasturrell deleted the thomasturrell-patch-2 branch November 14, 2025 22:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request attempts to update the Maven repository URLs in the pom.xml file to use new Sonatype endpoints for snapshot and release deployments. However, the updated URLs appear to have issues that may prevent successful artifact deployment.

Key Changes

  • Updated snapshot repository URL from s01.oss.sonatype.org to central.sonatype.com
  • Modified release repository URL to use ossrh-staging-api.central.sonatype.com endpoint

<repository>
<id>ossrh</id>
<url>https://s01.oss.sonatype.org/service/local/staging/deploy/maven2/</url>
<url>https://ossrh-staging-api.central.sonatype.com/service/local/</url>
Copy link

Copilot AI Nov 14, 2025

Choose a reason for hiding this comment

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

The release repository URL is incomplete. The staging deployment endpoint should include the full path. The current URL https://ossrh-staging-api.central.sonatype.com/service/local/ is missing the /staging/deploy/maven2/ path component.

The correct URL should be: https://ossrh-staging-api.central.sonatype.com/service/local/staging/deploy/maven2/

Suggested change
<url>https://ossrh-staging-api.central.sonatype.com/service/local/</url>
<url>https://ossrh-staging-api.central.sonatype.com/service/local/staging/deploy/maven2/</url>

Copilot uses AI. Check for mistakes.
<snapshotRepository>
<id>ossrh</id>
<url>https://s01.oss.sonatype.org/content/repositories/snapshots</url>
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
Copy link

Copilot AI Nov 14, 2025

Choose a reason for hiding this comment

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

The snapshot repository URL appears to be incorrect. According to Sonatype Central Publishing documentation, snapshots cannot be deployed to https://central.sonatype.com/repository/maven-snapshots/ (this is a browsing/download endpoint, not a deployment endpoint).

For the new Sonatype Central Publishing system, snapshots should typically be deployed to https://central.sonatype.com/api/v1/publisher/upload (or continue using the legacy OSSRH endpoint if you haven't migrated yet).

Additionally, note that the nexus-staging-maven-plugin configuration at line 190 still references the old URL https://s01.oss.sonatype.org/, which should also be updated to match the new endpoints if migrating to Sonatype Central Publishing.

Suggested change
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
<url>https://central.sonatype.com/api/v1/publisher/upload</url>

Copilot uses AI. Check for mistakes.
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.

2 participants