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

Don't allow time series names that start with underscore #75

Closed
happymcplaksin opened this issue Nov 21, 2013 · 2 comments
Closed

Don't allow time series names that start with underscore #75

happymcplaksin opened this issue Nov 21, 2013 · 2 comments

Comments

@happymcplaksin
Copy link

Right now the Admin UI lets you submit points for a time series that starts with underscore. In IRC jvshahid said it shouldn't allow you to do that.

Also, when querying via the Admin UI it silently strips underscore from the beginning of the time series name. For example, this query:

select * from _simple

will return results from the time series named 'simple' as if you had typed:

select * from simple

To be clear, I stumbled on this issue in a roundabout way--I don't need time series names that start with underscore :)

Thank you!

@jvshahid
Copy link
Contributor

Awesome. Thanks @happymcplaksin. I'll take a look at this very shortly.

@pauldix
Copy link
Member

pauldix commented Nov 21, 2013

Should be at the API level. It should throw a 400 with an error message that you can't start a time series name with an underscore, which should then be shown in the admin UI.

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