Skip to content

[GHSA-q3mw-pvr8-9ggc] Apache Tomcat Open Redirect vulnerability#7517

Open
hara-satoshi-ymr wants to merge 1 commit intohara-satoshi-ymr/advisory-improvement-7517from
hara-satoshi-ymr-GHSA-q3mw-pvr8-9ggc
Open

[GHSA-q3mw-pvr8-9ggc] Apache Tomcat Open Redirect vulnerability#7517
hara-satoshi-ymr wants to merge 1 commit intohara-satoshi-ymr/advisory-improvement-7517from
hara-satoshi-ymr-GHSA-q3mw-pvr8-9ggc

Conversation

@hara-satoshi-ymr
Copy link
Copy Markdown

Updates

  • Affected products

Comments
adding org.apache.tomcat.embed:tomcat-catalina

Copilot AI review requested due to automatic review settings April 28, 2026 07:11
@github-actions github-actions Bot changed the base branch from main to hara-satoshi-ymr/advisory-improvement-7517 April 28, 2026 07:12
Copy link
Copy Markdown
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

Updates the GHSA-q3mw-pvr8-9ggc advisory to expand the list of affected Maven coordinates for Apache Tomcat’s open redirect vulnerability.

Changes:

  • Bumped the advisory modified timestamp.
  • Added new affected entries for org.apache.tomcat.embed:tomcat-catalina across Tomcat 8.5.x, 9.0.x, 10.1.x, and 11.0.x version ranges.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +171 to +174
"package": {
"ecosystem": "Maven",
"name": "org.apache.tomcat.embed:tomcat-catalina"
},
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

The Maven coordinate org.apache.tomcat.embed:tomcat-catalina looks inconsistent with the coordinates used elsewhere in this repo: other Tomcat advisories track Catalina as org.apache.tomcat:tomcat-catalina (e.g. advisories/github-reviewed/2022/05/GHSA-3vx3-xf6q-r5xp/GHSA-3vx3-xf6q-r5xp.json:21) and embed artifacts as org.apache.tomcat.embed:tomcat-embed-core / tomcat-embed-*. If the intent is to cover the Catalina JAR, this should likely be org.apache.tomcat:tomcat-catalina (and updated consistently for each new affected entry), otherwise automated matching may never hit this package name.

Copilot uses AI. Check for mistakes.
Comment on lines +190 to +193
"package": {
"ecosystem": "Maven",
"name": "org.apache.tomcat.embed:tomcat-catalina"
},
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

Same issue as above: this affected entry uses org.apache.tomcat.embed:tomcat-catalina, but Tomcat Catalina is typically tracked as org.apache.tomcat:tomcat-catalina in this repo; please confirm the intended Maven coordinate and update accordingly so the advisory matches real dependencies.

Copilot uses AI. Check for mistakes.
Comment on lines +209 to +212
"package": {
"ecosystem": "Maven",
"name": "org.apache.tomcat.embed:tomcat-catalina"
},
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

Same issue as above: org.apache.tomcat.embed:tomcat-catalina is inconsistent with other advisories (which use org.apache.tomcat:tomcat-catalina). Please verify the correct Maven coordinate; otherwise this range will not apply to any real package.

Copilot uses AI. Check for mistakes.
Comment on lines +228 to +231
"package": {
"ecosystem": "Maven",
"name": "org.apache.tomcat.embed:tomcat-catalina"
},
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

Same issue as above: this entry uses the org.apache.tomcat.embed:tomcat-catalina coordinate; please verify and (if needed) change to the coordinate used for Catalina elsewhere in the repo (org.apache.tomcat:tomcat-catalina).

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