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.

Sign up for GitHub

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

Jump to bottom

Chap0x05 #3

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Chap0x05 #3

wants to merge 6 commits into from

Conversation

cucsecmodan
Copy link
Collaborator

No description provided.

@yumlii33
Copy link

yumlii33 commented Dec 1, 2022 •
edited
Loading

Accepted

Pros

  • 符合推荐的目录和分支结构
  • 在PR标题中体现了作业序号
  • 见名知意的文件命名
  • 有基本的配图和高亮标记
  • 实验报告结构规范、内容完整
  • Markdown使用规范，代码段高亮
  • 参考文献/资料引用标准规范
  • 作业提交及时
  • 懂得在作业中适时「植入自己的身份标识」来证明自己作业的原创性
  • 完成本次实验的全部必选和可选任务！实验报告完成的很用心 🎉
  • 对实验结果给出了自己的解释 👍
  • 对实验过程中遇到的问题和解决方法进行了记录 👍
  • 使用nc轻便开启TCP和UDP监听服务 👍
  • 使用iptables设置过滤规则 👍

Cons

  • 目录结构设置不正确，问题同实验报告chap0x04
  • tcp_connect_scan.py的代码未在实验报告中展示
  • 记得按时提交作业 ~

Recommends

  • 实验代码逻辑清晰、分析全面 👍，代码文件可以单独存放，参照老师给出的示例作业目录: 
    └── chap0x05
├── README.md
└── code
    ├── exp.py
    └── nginx.conf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cucsecmodan @yumlii33