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

[EXTERNAL] add strings to forbidden #2475

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

sagarishere
Copy link
Contributor

every exercise can currently be solved by using strings for example

strings bin | grep "modified"

to give the message required to complete the exercise.

However, that technique will teach us nothing about binary inspection, reverse engineering or binary manipulation.

Before starting, please choose the relevant pull request Labels, Reviewers, and Assignees

DEV-#### | SUP-#### | CON-####

Why?

It is important to learn concepts, not circumvent the exercise.

Solution Overview

Explicitly forbid strings command

Implementation Details

Added one line to README

Build Images

Please assign the label 📦 build If you want to re-build the following test images:

  • 💻 Sh
  • 🚀 JS
  • 🧩 DOM

@HarryVasanth HarryVasanth changed the title add strings to forbidden [EXTERNAL] add strings to forbidden Jul 8, 2024
@HarryVasanth HarryVasanth added the 📕 Cyber Security Cybersecurity label Nov 21, 2024
@HarryVasanth HarryVasanth added the 🗂️ docs Documentation label Dec 13, 2024
every exercise can currently be solved by using strings for example

` strings bin | grep "modified"` 

to give the message required to complete the exercise.

However, that technique will teach us nothing about binary inspection, reverse engineering or binary manipulation.
@HarryVasanth HarryVasanth merged commit e7171dd into 01-edu:master Dec 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📕 Cyber Security Cybersecurity 🗂️ docs Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants