Skip to content

Linux: Add first networking challenge: Firewall management#9

Merged
ryapric merged 22 commits intomainfrom
feature/add-dummy-web-app
Nov 28, 2023
Merged

Linux: Add first networking challenge: Firewall management#9
ryapric merged 22 commits intomainfrom
feature/add-dummy-web-app

Conversation

@ryapric
Copy link
Copy Markdown
Member

@ryapric ryapric commented Nov 5, 2023

Part of #8, but I also got carried away.

This PR:

  • Adds a basic firewall-management challenge using ufw. The expectations are outlined in challenge_5.md, where I would want them to use nmap to hit the DB server to check open ports, then fix the broken one. Though admittedly, they won't have to if they run e.g. ufw status. Note that I did also make sure port 8000 is open in main.tf as well.

  • Renames occurrences of step to challenge to better describe what they actually are.

  • Replaces printf calls that were pseudo-logs with ezlog

  • This also tightens up test.bats to include tests against pre-solved states.

  • Adds a simple GitHub Actions workflow to run linux/scripts/ci.sh

@ryapric ryapric mentioned this pull request Nov 5, 2023
@ryapric ryapric marked this pull request as ready for review November 13, 2023 03:20
@ryapric ryapric changed the title Add networking challenges Add first networking challenge: firewall management Nov 13, 2023
@ryapric ryapric self-assigned this Nov 13, 2023
@ryapric
Copy link
Copy Markdown
Member Author

ryapric commented Nov 13, 2023

@JDeBo this is ready for first-pass review now, just as a part of #8 and not the whole set of ideas yet.

Comment thread linux/instructions/step_5.md Outdated
Comment thread linux/Makefile Outdated
Comment thread linux/Vagrantfile
Comment thread linux/dummy-web-app-src/main.go Outdated
Comment thread linux/scripts/test.bats Outdated
Comment thread linux/dummy-web-app-src/Containerfile Outdated
@ryapric ryapric changed the title Add first networking challenge: firewall management Linux: Add first networking challenge: Firewall management Nov 19, 2023
@ryapric
Copy link
Copy Markdown
Member Author

ryapric commented Nov 19, 2023

@JDeBo ok I think we're in a good spot to merge this once we scrutinize challenge_5.md

@JDeBo
Copy link
Copy Markdown
Collaborator

JDeBo commented Nov 22, 2023

Other than those small fixes, lgtm, I've got it merged into my branch so pls no huge rebases lol

@JDeBo
Copy link
Copy Markdown
Collaborator

JDeBo commented Nov 22, 2023

Other than those small fixes, lgtm, I've got it merged into my branch so pls no huge rebases lol

You ever see this issue with the ezlog install in Vagrant?

    db: Cloning into '/usr/local/share/ezlog'...
    db: fatal: unable to access 'https://github.com/opensourcecorp/ezlog.git/': server certificate verification failed. CAfile: none CRLfile: none
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

@ryapric
Copy link
Copy Markdown
Member Author

ryapric commented Nov 24, 2023

Which small fixes do you mean?

Also no, but that looks like a proxy error -- especially if you're running on your work laptop, it might be that.

@JDeBo
Copy link
Copy Markdown
Collaborator

JDeBo commented Nov 25, 2023

Which small fixes do you mean?

Also no, but that looks like a proxy error -- especially if you're running on your work laptop, it might be that.

#9 (comment)

A couple small formatting things above like this

Comment thread .github/workflows/main.yaml
Comment thread linux/scripts/linux-workshop-admin.sh
Comment thread linux/scripts/test.bats
Comment thread linux/scripts/test.bats
Comment thread linux/instructions/challenge_5.md Outdated
@JDeBo
Copy link
Copy Markdown
Collaborator

JDeBo commented Nov 26, 2023

Which small fixes do you mean?
Also no, but that looks like a proxy error -- especially if you're running on your work laptop, it might be that.

#9 (comment)

A couple small formatting things above like this

Okay... so turns out I just don't understand how PR comments work in Github. Apparently I hadn't submitted them, hopefully you're able to view them now.

@ryapric
Copy link
Copy Markdown
Member Author

ryapric commented Nov 26, 2023

Which small fixes do you mean?
Also no, but that looks like a proxy error -- especially if you're running on your work laptop, it might be that.

#9 (comment)
A couple small formatting things above like this

Okay... so turns out I just don't understand how PR comments work in Github. Apparently I hadn't submitted them, hopefully you're able to view them now.

Yeah I was very confused but I see them now 😂 thank you!

@JDeBo
Copy link
Copy Markdown
Collaborator

JDeBo commented Nov 28, 2023

lgtm

@ryapric ryapric merged commit 045a587 into main Nov 28, 2023
@ryapric ryapric deleted the feature/add-dummy-web-app branch November 28, 2023 00:45
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