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

Update index.md #23

Open
wants to merge 69 commits into
base: master
Choose a base branch
from
Open

Update index.md #23

wants to merge 69 commits into from

Conversation

AdoX13
Copy link

@AdoX13 AdoX13 commented Jul 9, 2022

No description provided.

@AdoX13 AdoX13 requested a review from teodutu July 9, 2022 20:50
@AdoX13 AdoX13 requested a review from Ryx076 July 10, 2022 09:16
Copy link
Member

@teodutu teodutu left a comment

Choose a reason for hiding this comment

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

I gave you some initial feedback as inline comments. I also have some high-level observations:

  • s/number system/numeral system/g
  • The text in the image number_systems.svg is a bit off-centre. Make the figures larger so the text fits.
  • There's no need to leave an empty line after each sentence. Just write them on separate consecutive lines so they show up in the same paragraph. Only leave empty lines when you want to start a new paragraph.

data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Show resolved Hide resolved
data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Show resolved Hide resolved
data-representation/index.md Show resolved Hide resolved
data-representation/index.md Outdated Show resolved Hide resolved
Copy link
Member

@teodutu teodutu left a comment

Choose a reason for hiding this comment

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

I made a few extra suggestions.

data-representation/index.md Outdated Show resolved Hide resolved
data-representation/index.md Show resolved Hide resolved
@teodutu
Copy link
Member

teodutu commented Jul 11, 2022

@gabrielmocanu look at the rich diff of the README and you'll see the images. @Adoie yes, you'll have to squash these commits, but after you've added the challenges.

Copy link
Member

@teodutu teodutu left a comment

Choose a reason for hiding this comment

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

Follow the conventions in our template-public repo for storing challenges. You don't need README.md files for them, but:

  • rename challenge folders (eg: ASCII Art becomes ascii-art)
  • change solution script names to solution.py
  • use READNE.md instead of sol.md
  • rename all public files using lowercase letters and underscores (eg.: The Truth.txt becomes the_truth.txt)

Then I'll have another look.

data-representation/index.md Outdated Show resolved Hide resolved
AdoX13 and others added 12 commits July 12, 2022 13:39
…oject.txt to data-representation/activities/ascii-art/public/tommys_art_project.txt
…presentation/activities/ascii-art/sol/solution.py
…ata-representation/activities/infinity-hashes/sol/README.md
…h.txt to data-representation/activities/infinity-hashes/public/the_truth.txt
…epresentation/activities/froggified/sol/README.md
…t to data-representation/activities/froggified/public/froggified.txt
…ata-representation/activities/enconding-train/sol/solution.py
…ings.txt to data-representation/activities/enconding-train/public/my_encodings.txt
@AdoX13 AdoX13 requested review from teodutu and gabrielmocanu July 12, 2022 10:50
Copy link
Member

@teodutu teodutu left a comment

Choose a reason for hiding this comment

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

I like the challenges. The session text is also nice. Look at the README.md in the template repo. Use the same syntax for your solution READMEs:

# Activity Name: Solution

## Vulnerability

(This section may not fit. Omit it if it doesn't)

## Exploit

Describe your solution here.

And don't forget to add the challenges to the private repo as well. Follow the structure already present in the repo and in the template-internal repo.

Comment on lines +1 to +4
First, we observe that we have `binary` text, so we decode it and get the fake flag: `SSS{are_you_a_computer}`.
Next up, we see that the text looks like `ASCII`, so we decode it as `decimal`, thus getting the next fake flag: `SSS{nope_still_a_human}`.
After that, we see that all digits higher than 7 are gone, so we immediately think of `octal`, which will get us to the next fake flag: `SSS{you_look_like_you_know_linux_permissions}`.
We see a lot of `0x`s, so we know to decode from `hexadecimal` and we get to the last fake flag: `SSS{you_love_tic_tac_toe_huh}`.
Copy link
Member

Choose a reason for hiding this comment

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

Don't place flags in files. Let the students follow this tutorial to find them. Furthermore, this task can be solved with a script.

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.

3 participants