arya2004 / arya2004/fynewire

Consolidate Duplicate Filtering Logic

Open
#3 1 comment 0 reactions 1 assignee Claimed by @princyballabh View on GitHub
good first issue hacktoberfest hacktoberfest-accepted
Dominant language
Go
Stars
2
Forks
4
PR merge metrics
No merged PRs in 30d

Description

Both `filter.go` and `model/packet.go` have similar or identical `ciMatch` and filtering logic (`Apply`, `ApplyFilters`). This is redundant and makes maintenance harder.

**Task:**

* [ ] Move all filtering logic to the `model` package (`model/packet.go`).
* [ ] Remove redundant code from `filter.go` and delete the file.
* [ ] Update any imports/usages of the filtering logic in other files (e.g., `client.go`).

**Tip:**
Replace usages like `filter.ApplyFilters(...)` with `model.ApplyFilters(...)` and adjust arguments if needed.

**Why:**
This makes the codebase easier to maintain and reduces bugs due to logic duplication.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.