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

chap0x01 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chap0x01 #1

wants to merge 1 commit into from

Conversation

watermelon-milk
Copy link
Collaborator

实验一报告提交

@yumlii33
Copy link

Accepted

Pros

  • 符合推荐的目录和分支结构
  • 在PR标题中体现了作业序号
  • 见名知意的文件命名
  • 有基本的配图和高亮标记
  • 实验报告结构规范
  • Markdown使用规范，代码块高亮
  • 参考文献/资料引用标准规范
  • 作业提交及时
  • 实验完成度较高，网络连通性测试基本完成 🎉
  • 对实验过程中遇到的问题和解决方法进行了记录 👍

Cons

  • 代码段没有高亮
  • 注意用词准确，网关和攻击者主机配置的是NatNetwork，而不是NAT网卡。两者是有区别的，可以参考：6.2. Introduction to Networking Modes
  • 记得按时提交作业

Recommends

  • 建议绘制网络拓扑图
  • 完成本实验的要求，攻击者主机只需要设置一块网卡，选择「NAT网络」。多设置的两块Host-Only网卡似乎并没有用到

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
@watermelon-milk @yumlii33