-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[0.9.4.1] Panic due no protection on value containing '-' #4272
Comments
JulienChampseix
changed the title
[BUG] - Panic / No protection on value having only negatif character
[BUG] - Panic due no protection on value containing -
Sep 29, 2015
JulienChampseix
changed the title
[BUG] - Panic due no protection on value containing -
[BUG] - Panic due no protection on value containing '-'
Sep 29, 2015
thanks @JulienChampseix, I assume you are on 0.9.4.1? |
beckettsean
changed the title
[BUG] - Panic due no protection on value containing '-'
[0.9.4.1] Panic due no protection on value containing '-'
Sep 29, 2015
@beckettsean : InfluxDB version 0.9.4-rc1, branch master, commit 6d028f9 |
beckettsean
changed the title
[0.9.4.1] Panic due no protection on value containing '-'
[0.9.4] Panic due no protection on value containing '-'
Sep 30, 2015
I confirmed that this is still present in the 0.9.4.1 release. |
beckettsean
changed the title
[0.9.4] Panic due no protection on value containing '-'
[0.9.4.1] Panic due no protection on value containing '-'
Sep 30, 2015
Planned to be fixed in 0.9.5 ? |
@JulienChampseix it is unscheduled. |
Fixed by #4296 Thanks for the report @JulienChampseix |
thanks @otoolep ! 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
When i sending an malformed query write the influxdb server crashed (due non protection for this type of value).
Command curl :
curl -i -XPOST 'http://localhost:8086/write?db=brain.applications&u=test&p=test' --data-binary 'temperature,host=server02 value=-0.67,value2=-'
The value1 is well formatted but value2 only containing '-' which create a panic on the influxdb server (= crash)
Would be great to have a protection for something like that.
See bellow the stack crash :
The text was updated successfully, but these errors were encountered: