Skip to content

[MAIN] added json checks, fixed ip check logic #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 20 commits into from
May 12, 2025
Merged

[MAIN] added json checks, fixed ip check logic #5

merged 20 commits into from
May 12, 2025

Conversation

TheTeaCat
Copy link
Member

@TheTeaCat TheTeaCat commented May 12, 2025 •
edited
Loading

  • Implements only logging requests/responses with JSON content-types or payloads
  • Adds more filtering to packets based on src/dst IP
  • Bugfixes & additional logging

TheTeaCat and others added 14 commits May 9, 2025 16:36
@TheTeaCat
renames
a188b9e
@TheTeaCat
initial is_json implementation
bf56105
@TheTeaCat
use is_json in main
8af652f
@TheTeaCat
add ENABLE_ONLY_LOG_JSON env var to dev target in makefile
7cdfb1e
@TheTeaCat
ignore packets not destined for or originating from a service IP
9363737
@TheTeaCat
Doc ENABLE_ONLY_LOG_JSON env var
dd0a10e
@TheTeaCat
check both request and response payloads and headers
d8bc4cf
@rileyfiretail
Merge pull request #3 from FireTail-io/filter-packets
e8e9fda 
ignore packets not destined for or originating from a service IP
@TheTeaCat
add test case with +json
692bf93
@TheTeaCat
update isJson to check for +json suffix in content types
adc37a2
@TheTeaCat
Merge branch 'dev' into is-json
c7fdf66
@TheTeaCat
Implement & doc ONLY_LOG_JSON_MAX_CONTENT_LENGTH env var
ea270ae
@TheTeaCat
fix test cases with long json payloads to have valid json payloads
4f27e35
@rileyfiretail
Merge pull request #4 from FireTail-io/is-json
68e9d3c 
Is json
@TheTeaCat TheTeaCat requested a review from rileyfiretail May 12, 2025 09:48
@TheTeaCat TheTeaCat self-assigned this May 12, 2025
rileyfiretail
rileyfiretail previously approved these changes May 12, 2025
View reviewed changes
@rileyfiretail rileyfiretail merged commit 56dbb09 into main May 12, 2025
3 checks passed
@rileyfiretail rileyfiretail changed the title v0.2.0 [MAIN] added json checks, fixed ip check logic May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rileyfiretail rileyfiretail rileyfiretail approved these changes

Assignees

@TheTeaCat TheTeaCat

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TheTeaCat @rileyfiretail