Skip to content

fix(internet): an MH CGA parameter with a raw extension in its data model, or no public_key_test, fails to build #2725

Closed

Description

@JarryShaw

Found while fixing #2723. Both reproduce on main at 596dcfaf8; neither involves an empty item.

  • A raw CGA extension inside a data model fails. A CGAParameter data model carrying any non-empty raw extension, even a whole valid one, raises AttributeError in _read_cga_extensions. The same extension given through a schema builds fine. The likely cause is that _passes_raw does not look inside a data model, so the made header is never re-parsed and read() meets the raw octets. This is not yet confirmed.
  • A CGA parameter schema built without public_key_test fails on build with KeyError: 'public_key_test'. hasattr reports the field as present, so the maker never sets it.

Whatever parses must also build, and a caller value that cannot build must raise ProtocolError.

Activity

  1. added this to the PyPCAPKit v1.6 milestone on Nov 3, 2026
  2. added
    bugIssues reporting a defect (set by the bug report template; a default, not an assessment)
    pendingAccepted and unstarted - nobody is on it and nothing blocks it
    on Nov 3, 2026
  3. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    Filed and accepted; nothing has started yet. The host is loaded, so a worker goes on this at the next sweep that has headroom. It also touches mh.py, which #2723 changes, so it should start after #2723 is in. pending.

  4. moved this to Pending in PyPCAPKiton Nov 3, 2026
  5. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    Now blocked until #2723 is in, since both edit the MH makers in mh.py. The host has headroom again, so this starts as soon as #2723 lands.

  6. 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 Nov 3, 2026
  7. moved this from Pending to Blocked in PyPCAPKiton Nov 3, 2026
  8. added
    blockedDeferred pending another issue or decision; see the last comment for what unblocks it
    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 Nov 3, 2026
  9. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    Unblocked: #2723 merged as 468f9c219. A worker is on this now, based on that sha. wip.

  10. moved this from Blocked to WIP in PyPCAPKiton Nov 3, 2026
  11. JarryShaw commented on Nov 3, 2026

    @JarryShaw
    OwnerAuthor

    The PR is #2732.

  12. removed
    wipWork in flight - a covering PR is open or an agent is actively on it
    on Nov 3, 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

    bugIssues reporting a defect (set by the bug report template; a default, not an assessment)

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions