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

Chap0x01 #1

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

Chap0x01 #1

wants to merge 3 commits into from

Conversation

GaoSinan
Copy link
Collaborator

@GaoSinan GaoSinan commented Jan 6, 2023

提交chap0x01

@pigeoner
Copy link

pigeoner commented Jan 7, 2023

Accepted

Pros

  • 在Pr标题中体现了作业序号
  • 推荐的目录结构
  • 见名知意的文件命名
  • 实验报告结构规范内容完整
  • 作业中有适当的「身份标识」证明作业原创性
  • 有基本的配图与高亮标记,直接在配图中对高亮标记进行文字标注
  • 在实验报告末总结了遇到的问题与解决方案
  • 参考文献/资料引用标准规范

Cons

  • 没有任何个人身份标识,如何证明作业是自己完成的?
  • 你的攻击者主机和网关配置的是 NAT 网卡而不是 NAT 网络NAT 模式下虚拟机之间并不互通,网关 ping 10.0.2.15 也只是当成本地地址处理了
    • 注意比较 ping 不同虚拟机的响应时间
    • 另外,同一局域网内也不会允许多个相同 IP 地址存在
  • 作业提交太晚
  • 没有参考资料吗?
  • 仔细阅读一下作业要求,markdown的格式也是评分标准之一,GitHub上图片渲染出现问题就应该从markdown本身格式上去找问题,而不是上传一个pdf就能解决一切。你的图片渲染失败是因为路径分隔符用了\而不是/

Recommends

  • 拓扑图最好自己绘制,加深理解

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