Skip to content

Test 404 delegation in @fedify/h3 #860

Closed

Description

@dahlia

@fedify/h3 treats a 404 Fedify response as a signal that H3 should continue routing. That delegation path lets federation routes coexist with normal app routes.

Add a test in packages/h3/src/index.test.ts that verifies integrateFederation() does not call event.respondWith() when Fedify returns 404.

Suggested check: mise run check-each h3.

Activity

  1. added theissue type on Jun 30, 2026
  2. updated Priority Low and Effort Low on Jun 30, 2026
  3. Jae-Hyuk-Jang commented on Aug 8, 2026

    @Jae-Hyuk-Jang
    Contributor

    Can I be assigned this issue?

  4. github-actions commented on Aug 23, 2026

    @github-actions
    Contributor

    @Jae-Hyuk-Jang This issue has been assigned for over two weeks without updates. Please provide a status update, or unassign yourself if you're unable to continue working on it.

  5. Jae-Hyuk-Jang commented on Aug 23, 2026

    @Jae-Hyuk-Jang
    Contributor

    @Jae-Hyuk-Jang This issue has been assigned for over two weeks without updates. Please provide a status update, or unassign yourself if you're unable to continue working on it.

    Sorry for the slow update — still working on this. #859, #860, #861, and #862
    all touch the same integrateFederation()/onError() code path in
    packages/h3/src/index.ts, so I'm planning to cover all four in one PR with
    a new packages/h3/src/index.test.ts (the package currently has no test
    setup at all, so I'm scaffolding that too). Will open a PR soon.

  6. added a commit that references this issue on Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

Fields

Priority

Low

Effort

Low

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions