Skip to content

Conversation

ryanmurf
Copy link

Updates

  • Affected products

Comments
struts 1.1 - 1.2.9 is under the groupId struts and the artifactId is struts.
struts >=1.3 the groupId is org.apache.struts and artifactId is struts-core.

@Copilot Copilot AI review requested due to automatic review settings October 12, 2025 15:39
Copy link

@Copilot 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 PR updates a GitHub Security Advisory for CVE-2015-0899 to correctly specify affected Apache Struts packages by separating the version ranges for two different Maven coordinates that were used for different Struts versions.

  • Updates the modified timestamp to reflect recent changes
  • Splits affected package definitions to handle different Maven groupId/artifactId patterns
  • Adds explicit coverage for the legacy struts:struts package coordinate

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"type": "ECOSYSTEM",
"events": [
{
"introduced": "1.1"
Copy link

Copilot AI Oct 12, 2025

Choose a reason for hiding this comment

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

The new struts:struts package entry is missing an upper bound version range. According to the PR description, struts 1.1 - 1.2.9 uses the struts:struts coordinate, so this range should include a last_affected field set to 1.2.9 to properly define the affected version range.

Suggested change
"introduced": "1.1"
"introduced": "1.1"
},
{
"last_affected": "1.2.9"

Copilot uses AI. Check for mistakes.

@github-actions github-actions bot changed the base branch from main to ryanmurf/advisory-improvement-6306 October 12, 2025 15:40
@ryanmurf ryanmurf closed this Oct 12, 2025
@github-actions github-actions bot deleted the ryanmurf-GHSA-cvvx-r33m-v7pq branch October 12, 2025 15:41
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