Skip to content

[WIP] Java : Add support for detecting Template Injections in Java. #3353

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 24, 2020

This is a WIP. Please don't merge.

-- EDIT --
I have only included files for Velocity and Freemarker but I am trying to add Pebble too.

@pwntester
Copy link
Contributor

Hey @porcupineyhairs, it is great you are working on a SSTI query. I wrote this query that I used for some research but didnt get the time to make a proper PR. Since you are already working in this query you may want to consider the sinks in my query.

@ghost
Copy link
Author

ghost commented May 22, 2020

@pwntester Your query is definitely useful. I will include it with my PR and push the changes over the weekend.

@ghost
Copy link
Author

ghost commented May 25, 2020

@aschackmull @pwntester Does java have something like debstubber or I am to manually stub the dependencies?

@aschackmull
Copy link
Contributor

Not to my knowledge, unfortunately. So far, we've been adding stubs manually.

@ghost ghost force-pushed the TemplateInjection branch from e71edb1 to ce15b1b Compare May 27, 2020 21:08
@ghost ghost force-pushed the TemplateInjection branch from ce15b1b to 91b1350 Compare June 26, 2020 17:24
@adityasharad adityasharad changed the base branch from master to main August 14, 2020 18:34
@ghost ghost mentioned this pull request May 20, 2021
@ghost
Copy link
Author

ghost commented May 20, 2021

I am closing this as the main has moved during the time this was pending.

Instead I am have opened a new PR #5935 with the necessary changes.

@ghost ghost closed this May 20, 2021
@ghost ghost deleted the TemplateInjection branch May 20, 2021 20:50
This pull request was closed.
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.

2 participants