Skip to content

feat(templated):cve-2013-2251 - #868

Open
sanjaymahajan14 wants to merge 8 commits into
google:masterfrom
sanjaymahajan14:feature/cve-2013-2251
Open

feat(templated):cve-2013-2251#868
sanjaymahajan14 wants to merge 8 commits into
google:masterfrom
sanjaymahajan14:feature/cve-2013-2251

Conversation

@sanjaymahajan14

Copy link
Copy Markdown
Contributor

I have created a plugin for CVE-2013-2251. Please review and approve.
I have also created testbed for this PR.
google/security-testbeds#232

@robert-doyensec robert-doyensec left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Change to recommendation, removed fingerprint because it was specific to the vulnerability demo and doesn't appear easy to do on struts since every page appears customizable, and changed the probe_s2_016_ognl and variables logic to detect correctly (appeared to have nuclei variable).

Comment thread templated/templateddetector/plugins/cve/2013/ApacheStruts_CVE_2013_2251.textproto Outdated
Comment thread templated/templateddetector/plugins/cve/2013/ApacheStruts_CVE_2013_2251.textproto Outdated
Comment thread templated/templateddetector/plugins/cve/2013/ApacheStruts_CVE_2013_2251.textproto Outdated
Comment thread templated/templateddetector/plugins/cve/2013/ApacheStruts_CVE_2013_2251.textproto Outdated
Comment thread templated/templateddetector/plugins/cve/2013/ApacheStruts_CVE_2013_2251.textproto Outdated
Comment thread templated/templateddetector/plugins/cve/2013/ApacheStruts_CVE_2013_2251.textproto Outdated
sanjaymahajan14 and others added 7 commits August 17, 2026 15:53
Co-authored-by: Robert Dick <robert@doyensec.com>
Co-authored-by: Robert Dick <robert@doyensec.com>
Co-authored-by: Robert Dick <robert@doyensec.com>
Co-authored-by: Robert Dick <robert@doyensec.com>
Co-authored-by: Robert Dick <robert@doyensec.com>
Co-authored-by: Robert Dick <robert@doyensec.com>
Co-authored-by: Robert Dick <robert@doyensec.com>

@robert-doyensec robert-doyensec left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM - Approved
This can be merged along with google/security-testbeds#232.

Reviewer: Robert, Doyensec
Plugin: Apache Struts2 S2-016 DefaultActionMapper OGNL Injection
Drawbacks: None.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants