Skip to content
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

SIP sidestepping for tomcat, using force accessing of fields #192

Merged
merged 14 commits into from
Nov 15, 2023

Conversation

t4lz
Copy link
Member

@t4lz t4lz commented Nov 10, 2023

This is a very hacky way to solve that problem, but the only solution I got to work so far.
We force change a field accessibility and then set it.

So far the SIP-sidestepping works, and a browser window pops up. By default the url is localhost:8080, which works. We could think about trying to find out the remote external IP and port and change it to that, but I assume it's not required for now - the user can set the remote url in the launch configuration.

Stopping still requires pressing the button twice, but there is no error message.

@t4lz t4lz force-pushed the tommy-override-default branch from 5fbf84d to 7ac36da Compare November 10, 2023 21:49
@t4lz t4lz force-pushed the tommy-override-default branch from 7ac36da to 2958eab Compare November 15, 2023 10:27
@t4lz t4lz force-pushed the tommy-override-default branch from 2958eab to 70eb26f Compare November 15, 2023 10:29
@t4lz t4lz marked this pull request as ready for review November 15, 2023 13:27
@aviramha
Copy link
Member

Great job!

@t4lz t4lz enabled auto-merge November 15, 2023 18:46
@t4lz t4lz added this pull request to the merge queue Nov 15, 2023
Merged via the queue into metalbear-co:main with commit 8ddf0a7 Nov 15, 2023
5 checks passed
@t4lz t4lz deleted the tommy-override-default branch November 15, 2023 18:57
This was referenced Nov 20, 2023
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