Skip to content

fix: register /api/dissectors route to fix 404 on settings page#44

Merged
tito merged 2 commits intomainfrom
fix/register-dissectors-api-route
Apr 9, 2026
Merged

fix: register /api/dissectors route to fix 404 on settings page#44
tito merged 2 commits intomainfrom
fix/register-dissectors-api-route

Conversation

@tito
Copy link
Copy Markdown
Contributor

@tito tito commented Apr 9, 2026

Summary

  • The DissectorsListHandler was already defined in endpoint_rules.go but was never registered as a route in router.go
  • This caused /api/dissectors to return 404, leaving the decoder dropdown on the settings page stuck on "loading..."
  • Adds the missing GET /api/dissectors route

Test plan

  • Open the settings page and verify the decoder dropdown loads with available dissectors
  • Verify endpoint rule creation still works with the decoder selection

tito added 2 commits April 9, 2026 11:47
The DissectorsListHandler was defined in endpoint_rules.go but never
wired to a route, causing the settings page decoder section to stay
stuck on "loading...".
@tito tito merged commit ce55a0a into main Apr 9, 2026
3 checks passed
@tito tito deleted the fix/register-dissectors-api-route branch April 9, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant