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.

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

Fix headers escaping and parsing #531

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

AsafMah
Copy link
Collaborator

@AsafMah AsafMah commented Apr 21, 2024

Changes the tracing format to be always allowed in headers, and easier to reason about:

  • Only allowed chars: [a-zA-Z0-9_.-()] (Note: not using \w because we don't want non-ascii chars)
  • Escape every part of the header, even when using set_connector_details
  • Remove the brackets around escaped fields, as they are not needed to parse anymore. So - Parsing is now simply splitting by |, and then you have pairs of key:value.
  • Limit the length of each field to 100

So old: Kusto.Café:{1 . 0}|App.{Something}:{weird version}
New: Kusto.Caf_:1_._0|App.Something:weird_versio

…r to reason about:

- Only allowed chars: [a-zA-Z0-9_.\-()] (Note: not using \w because we don't want non-ascii chars)
- Escape every part of the header, even when using set_connector_details
- Remove the brackets around escaped fields, as they are not needed to parse anymore. So
- Parsing is now simply splitting by |, and then you have pairs of key:value.
- Limit the length of each field to 100

So old:
Kusto.Café:{1 . 0}|App.{Something}:{weird version}
New:
Kusto.Caf_:1_._0|App.Something:weird_version
Copy link

github-actions bot commented Apr 21, 2024

Test Results

    6 files  ± 0      6 suites  ±0   19m 41s ⏱️ - 1m 36s
  290 tests + 2    264 ✅ + 2   26 💤 ±0  0 ❌ ±0 
1 740 runs  +12  1 584 ✅ +12  156 💤 ±0  0 ❌ ±0 

Results for commit af29e38. ± Comparison against base commit a436993.

♻️ This comment has been updated with latest results.

…r to reason about:

- Only allowed chars: [a-zA-Z0-9_.\-()] (Note: not using \w because we don't want non-ascii chars)
- Escape every part of the header, even when using set_connector_details
- Remove the brackets around escaped fields, as they are not needed to parse anymore. So
- Parsing is now simply splitting by |, and then you have pairs of key:value.
- Limit the length of each field to 100

So old:
Kusto.Café:{1 . 0}|App.{Something}:{weird version}
New:
Kusto.Caf_:1_._0|App.Something:weird_version
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