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

feat: add ZAP package #401

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

feat: add ZAP package #401

wants to merge 3 commits into from

Conversation

noahfraiture
Copy link

This is a remake of #390, I though it was ok but wrong branch and unsigned, it was thus easier to remake a new branch. Sorry for the wait, I didn't notice the requirement for the sign on the commit

Description

This PR add Zed Attack Proxy as requested in this #381 .


This is my first PR here and I did not succeed to test. When trying to exegol install local I got stuck at [*] Step 14/24 : RUN ./entrypoint.sh package_base
It should work since the PR is very small but if someone can confirm before a merge, it would be great

@QU35T-code QU35T-code self-assigned this Oct 18, 2024
@QU35T-code QU35T-code added new tool(s) This adds one or multiple tools to Exegol under review Time to think about the need / implementation labels Oct 18, 2024
Copy link
Member

@QU35T-code QU35T-code left a comment

Choose a reason for hiding this comment

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

Hi @noahfraiture,

Thanks for taking the time to suggest this addition !

sources/install/package_web.sh Outdated Show resolved Hide resolved
@QU35T-code QU35T-code added the waiting for additional changes Further changes are requested label Oct 18, 2024
Copy link
Member

@QU35T-code QU35T-code left a comment

Choose a reason for hiding this comment

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

Last modifications and all should be good !

sources/install/package_web.sh Outdated Show resolved Hide resolved
sources/install/package_web.sh Outdated Show resolved Hide resolved
@QU35T-code
Copy link
Member

Also, can you clean the /tmp dir ? rm -rf /tmp/ZAP_${zap_version} and rm /tmp/zap.tar.gz

@noahfraiture
Copy link
Author

Yep done

sources/install/package_web.sh Outdated Show resolved Hide resolved
sources/install/package_web.sh Outdated Show resolved Hide resolved
Copy link
Member

@QU35T-code QU35T-code left a comment

Choose a reason for hiding this comment

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

Last changes ! :)

sources/install/package_web.sh Outdated Show resolved Hide resolved
@QU35T-code QU35T-code added waiting for review A review is requested and removed waiting for additional changes Further changes are requested labels Oct 21, 2024
rm /tmp/zap.tar.gz
add-aliases zap
add-history zap
add-test-command "which zap"
Copy link
Member

Choose a reason for hiding this comment

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

Not a big fan of that test command to be honest, is there any way to test for the install?

Copy link
Author

Choose a reason for hiding this comment

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

The problem is that there's no command available through the CLI. Any command start the GUI

@ShutdownRepo ShutdownRepo added waiting for additional changes Further changes are requested and removed waiting for review A review is requested under review Time to think about the need / implementation labels Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new tool(s) This adds one or multiple tools to Exegol waiting for additional changes Further changes are requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can you add ZAP to exegol please ? (zed attack proxy)
3 participants