forked from kbandla/dpkt
-
Notifications
You must be signed in to change notification settings - Fork 1
Weekly Report III (June 11~June 17)
Hao Sun edited this page Jun 18, 2015
·
2 revisions
My main job for this week is to follow up the issues and make deeper code inspections during that process. The detailed issue status assigned to me is as follows.
-
Issue #117: dpkt http wrong parse_headers logic. Sent out PR #181, #183 and #191. Now the bug has been fixed and related unit tests were added. All of the changes have been merged to
master
branch. - Issue #164: Made http.py return ordered headers. Sent out PR #188. However, according to our discussion, this feature might not be a supposed feature of dpkt. The related PR has been closed.
-
Issue #156: IP total length header field not stored correctly. At first, I plan to add some setters with
@property
decreator, however this implementation would affect the existing client code. I think this is an issue need to be discussed. - Issue #81: dpkt producing a bad TCP checksum. In my test, the checksum of TCP packet is correct. It seems that the problem has been fixed. I think this issue is ready to be closed.
In summary, the issue status for my side is as follows.
- Solve more issues and fix related bugs.
- Dive into the details of
dpkt
modules and related implementation. - Get started on is the documentation for readthedocs.