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

what to do about special charaters in field values? #4

Closed
alsuren opened this issue Sep 8, 2024 · 3 comments
Closed

what to do about special charaters in field values? #4

alsuren opened this issue Sep 8, 2024 · 3 comments

Comments

@alsuren
Copy link
Contributor

alsuren commented Sep 8, 2024

I am currently collecting user agent information over at cargo-bins/cargo-quickinstall#165 (review) and I managed to end up with some a fun Tag Key of like Gecko) Chrome/70.0.3538.67 Safari/537.36,version (see screenshot from the latest influxdb cloud).

Screenshot 2024-09-08 at 12 06 54

I'm assuming something is not being escaped correctly somewhere? I'm still digging into it.

https://docs.influxdata.com/influxdb/v1/write_protocols/line_protocol_tutorial/#special-characters-and-keywords says to use \ for escaping everything. It seems like most other user agent strings are correctly reported, so I'm going to try writing a unit test for this and see where the problem is.

@NobodyXu
Copy link

NobodyXu commented Sep 8, 2024

Maybe it's too long that it doesn't show the full string?

Or is it truncated somewhere?

@alsuren
Copy link
Contributor Author

alsuren commented Sep 8, 2024

I think the problem is more likely to be on our end. Sorry for the spam. Will re-open if it's a library problem once I've narrowed it down.

@alsuren alsuren closed this as completed Sep 8, 2024
@alsuren alsuren reopened this Sep 8, 2024
@alsuren alsuren mentioned this issue Sep 8, 2024
@ijagberg
Copy link
Owner

I have published version 2.1.0, which includes the changes in #5

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

No branches or pull requests

3 participants