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

Tag support characters escaped #33

Open
Jishun opened this issue May 9, 2021 · 0 comments
Open

Tag support characters escaped #33

Jishun opened this issue May 9, 2021 · 0 comments

Comments

@Jishun
Copy link

Jishun commented May 9, 2021

according to https://graphite.readthedocs.io/en/latest/tags.html
tag format is my.series;tag1=value1;tag2=value2
however nginx-statsd escapes too many kinds of characters ,
to get tag compatibility
need to replace the escape mask from line:

0xfc00bfff, /* 1111 1100 0000 0000 1011 1111 1111 1111 */

to below code

		                    /* ?>=< ;:98 7654 3210  /.-, +*)( '&%$ #"!  */
	//	0xfc00bfff, /* 1111 1100 0000 0000  1011 1111 1111 1111 */
		0xD400BFFF, /* 1101 0100 0000 0000  1011 1111 1111 1111 */

this repo looks out of maintenance for a long time, posting the patch here so that everyone can see it

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

1 participant