Skip to content

feat: switch from exempt routes to relevant routes approach#247

Merged
twilson63 merged 1 commit intomainfrom
feat/pattern_matching_non_exempt
Apr 25, 2025
Merged

feat: switch from exempt routes to relevant routes approach#247
twilson63 merged 1 commit intomainfrom
feat/pattern_matching_non_exempt

Conversation

@PeterFarber
Copy link
Copy Markdown

This commit changes the router's preprocessing logic from a blacklist to a whitelist approach:

  • Rename is_exempt to is_relevant and invert the preprocessing logic
  • Replace DEFAULT_EXEMPT_ROUTES with DEFAULT_RELAVANT_ROUTES
  • Only process ~process@1.0 paths, instead of exempting specific paths
  • Remove unnecessary debug logging statements
  • Fix tilde escape sequence in binary:split pattern

This approach makes message processing more predictable by explicitly defining what should be processed rather than what should be exempted.

This commit changes the router's preprocessing logic from a blacklist to a whitelist approach:

- Rename is_exempt to is_relevant and invert the preprocessing logic
- Replace DEFAULT_EXEMPT_ROUTES with DEFAULT_RELAVANT_ROUTES
- Only process ~process@1.0 paths, instead of exempting specific paths
- Remove unnecessary debug logging statements
- Fix tilde escape sequence in binary:split pattern

This approach makes message processing more predictable by explicitly defining
what should be processed rather than what should be exempted.
@twilson63 twilson63 merged commit bbdf5e5 into main Apr 25, 2025
@twilson63 twilson63 deleted the feat/pattern_matching_non_exempt branch April 25, 2025 14:23
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.

2 participants