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

Article "Firewall" #89

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

Article "Firewall" #89

wants to merge 3 commits into from

Conversation

fabiandudler
Copy link
Collaborator

Added article on "Firewall"

@fabiandudler fabiandudler changed the title Firewall Article "Firewall" Oct 29, 2024
@Ajorian
Copy link
Collaborator

Ajorian commented Nov 8, 2024

I personally like the metaphor you used to talk about firewalls. But, from my perspective, this works only for the case of stateless firewalls. I think it would be better if you talk also a little bit about state-full firewalls.

@evos96
Copy link
Collaborator

evos96 commented Nov 13, 2024

External feedback:

negative aspect(s) pointed out:
-It was not clear. (2)
-The article was too technical. (2)
-The structure was confusing (1)
-I didn't understand the purpose. (3)

positive aspect(s) pointed out:
-The information was easy to understand. (6)
-Good balance between technical depth and readability.(2)
-I stayed engaged throughout.(2)
-I understood the purpose (3)

Did the drawing help you to understand the meaning of the article?
yes (9)
no (4)

Did you notice any factual errors or inaccuracies in the article? If Yes, which part?
-It was too clear

Additional comments?
-Nice
-You mention burning wall and in the graphic there is a robot - match the graphic to the text better.

@carinaschrenk
Copy link
Collaborator

Internal Feedback Round 1B 14.11

Thoughts

  • The illustration is not related to the topic.
  • The concept Firewall is not linked to the title; the last paragraph explains what the firewall is, but it doesn’t relate to the rest of the text.

Ideas

  • The article could benefit from an explanation of stateful, it only mentions stateless.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants