Skip to content

fix(engines): pyshark engine counts a systemd Journal Export Block as a frame #1515

Closed

Description

@JarryShaw

The #1508 worker measured examples/captures/test.pcapng: engine='pyshark' gives 6 frames, while the default engine gives 5. tshark numbers the Journal Export Block as frame 5, and read_frame takes every tshark frame (pcapkit/foundation/engines/pyshark.py:243-249). I have not reproduced this myself.

Expected: frame counts agree with the default engine. Skip tshark frames that come from a non-packet block.

Correction: in CI this case first crashed on a bug in #1508's own harness, which is now fixed, not on the TCP-flags bug. test.pcapng has no TCP flows. Whether CI's tshark 4.2.2 also counts the block will show on #1508's parity jobs. 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. 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