Skip to content

fix: serialization of \n, \r and \t to Line Protocol, = is valid sign for measurement name #106

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

Merged
merged 2 commits into from
Jun 29, 2020

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Jun 25, 2020

Related to influxdata/influxdb-client-java#127

  • serialization of \n, \r and \t to Line Protocol
  • = is valid sign for measurement name
  • CHANGELOG.md updated
  • Rebased/mergeable
  • Tests pass
  • Sign CLA (if not already signed)

@bednar bednar requested a review from rolincova June 25, 2020 11:05
@bednar bednar added the bug Something isn't working label Jun 25, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jun 25, 2020

Codecov Report

Merging #106 into master will decrease coverage by 0.03%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   15.67%   15.64%   -0.04%     
==========================================
  Files          50       50              
  Lines        4989     5000      +11     
  Branches      252      253       +1     
==========================================
  Hits          782      782              
- Misses       4187     4197      +10     
- Partials       20       21       +1     
Impacted Files Coverage Δ
Client/Writes/PointData.cs 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5c0e54...559cbb8. Read the comment docs.

@bednar bednar changed the title fix: serialization of \n, \r and \t to Line Protocol fix: serialization of \n, \r and \t to Line Protocol, = is valid sign for measurement name Jun 26, 2020
@bednar bednar requested a review from rolincova June 29, 2020 06:55
@bednar bednar added this to the 1.10.0 milestone Jun 29, 2020
@bednar bednar merged commit 70eafdf into master Jun 29, 2020
@bednar bednar deleted the fix/escaping-characters branch June 29, 2020 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants