Skip to content

Make module work #5

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Make module work #5

wants to merge 3 commits into from

Conversation

AmaliaI12
Copy link

No description provided.

Modify python version in Dockerfile to 3.11 to use the tomllib module
Fix typo in cli.py

Signed-off-by: Amalia Ionescu <[email protected]>
Handeled `StatusCode.UNAVAILABLE` error with a retry loop and 1s delay
Added requests and angr to `pyproject.toml` to fix dependency issues

Signed-off-by: Amalia Ionescu <[email protected]>
@razvand razvand requested a review from ClaraStefania March 14, 2025 12:27
@razvand razvand self-assigned this Mar 14, 2025
Copy link

@razvand razvand left a comment

Choose a reason for hiding this comment

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

See initial review on formatting.

```

This is a problem that may occur in certain situations, preventing Poetry from getting packages.

Copy link

Choose a reason for hiding this comment

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

Remove extra blank line.

README.md Outdated
5. Build the arguments' adapter via `cd others/argv_adapter && make`.
- Changing the Docker socket permissions (unsecure approach) via `sudo chmod 777 /var/run/docker.sock`.


Copy link

Choose a reason for hiding this comment

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

Remove extra blank line.

```

you should get a path ending with `.venv/bin/poetry`.

Copy link

Choose a reason for hiding this comment

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

Remove extra blank line.

```console
poetry install --only main
```
1. Build the Docker image:
Copy link

Choose a reason for hiding this comment

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

Suggested change
1. Build the Docker image:
1. Build the Docker image:

@@ -69,7 +117,7 @@ The exploiter could generate an exploit with the outcome of DENIAL_OF_SERVICE an
00000000: 61 61 61 61 61 61 61 61 aaaaaaaa
```

#### Help
#### Get help
Copy link

Choose a reason for hiding this comment

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

Suggested change
#### Get help
#### Get Help

docker build --tag zeratool_lib -f docker/Dockerfile.zeratool_lib .
```

1. Ensure the Docker API is accessible by:
Copy link

Choose a reason for hiding this comment

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

Suggested change
1. Ensure the Docker API is accessible by:
1. Ensure the Docker API is accessible by:

@AmaliaI12 AmaliaI12 force-pushed the make_module_work branch from 446a8bc to 2fcd3c9 Compare May 2, 2025 07:59
Signed-off-by: Amalia Ionescu <[email protected]>
@AmaliaI12 AmaliaI12 force-pushed the make_module_work branch from 2fcd3c9 to 0f88a2a Compare May 2, 2025 08:01
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