Skip to content

fix(toolkit): pyshark TCP trace crashes because tshark 4.x reports flags as 'True'/'False' #1514

Closed

Description

@JarryShaw

pcapkit/toolkit/pyshark.py:415-417 parses bool(int(tcp.flags_syn)), and likewise for flags_fin and flags_reset. tshark 4.x reports these fields as 'True'/'False'. I measured repr(p.tcp.flags_syn) == "'True'" with tshark 4.6.9, and CI runs 4.2.2. The first traced TCP frame therefore raises ValueError: invalid literal for int() with base 10: 'True'.

Per the #1508 worker, this fails every aspect on all 9 captures that have TCP flows.

Expected: accept both the '1'/'0' and the 'True'/'False' forms.

Found by #1508 (Gap(0)). Refs #1202.

Activity

  1. added
    fixPull requests that fix a defect (fix: subject prefix)
    pendingAccepted and unstarted - nobody is on it and nothing blocks it
    on Oct 9, 2026
  2. JarryShaw commented on Oct 9, 2026

    @JarryShaw
    OwnerAuthor

    Blocked until #1508 merges. That PR adds this defect's known-failure row in tests/foundation/engines/test_engine_agreement_runtime.py, and a fix has to delete that row, so the fix waits for the row to exist on main.

  3. added
    blockedDeferred pending another issue or decision; see the last comment for what unblocks it
    and removed
    pendingAccepted and unstarted - nobody is on it and nothing blocks it
    on Oct 9, 2026
  4. moved this to Blocked in PyPCAPKiton Oct 9, 2026
  5. JarryShaw commented on Oct 9, 2026

    @JarryShaw
    OwnerAuthor

    Unblocked: #1508 has merged, so this defect's known-failure row is on main. A worker is on the fix, and the fix PR deletes the row.

  6. added
    wipWork in flight - a covering PR is open or an agent is actively on it
    and removed
    blockedDeferred pending another issue or decision; see the last comment for what unblocks it
    on Oct 9, 2026
  7. moved this from Blocked to WIP in PyPCAPKiton Oct 9, 2026
  8. removed
    wipWork in flight - a covering PR is open or an agent is actively on it
    on Oct 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    fixPull requests that fix a defect (fix: subject prefix)

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions