Skip to content
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

remove unused fields from network information #466

Closed
vigneshshanmugam opened this issue Mar 8, 2022 · 0 comments · Fixed by #467
Closed

remove unused fields from network information #466

vigneshshanmugam opened this issue Mar 8, 2022 · 0 comments · Fixed by #467
Assignees
@vigneshshanmugam
Labels
bug Something isn't working

Comments

@vigneshshanmugam
Copy link
Member

vigneshshanmugam commented Mar 8, 2022 •
edited by zube bot
Loading

  • We need a allowlist of fields that can be set as part of the journey/network_info event rather than creating the JSON with all the available fields from CDP. This would also avoid the max buffer limit that gets triggered and causes issues like reporter stream buffer is not emptied in all cases #446

  • Reason being it leads of explosion of data due to the field Subject Alternative Name list. A single request from recaptcha.net has a SAN list of 500KiB of data

  • We need this as an interim solution until we get this PR back feat: use events from PW for network data #372 and move to newer versions of PW.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vigneshshanmugam vigneshshanmugam

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: keep only necessary fields for network info vigneshshanmugam/synthetics
1 participant
@vigneshshanmugam