supabase / supavisor / ea562515efc10a0fb32a009539afb7b5cb980143-PR-580

47%

main: 65%

Pull #580

github

fix: more correct name validation

PostgreSQL requires that the DB name is a valid identifier

https://www.postgresql.org/docs/current/sql-syntax-lexical.html#SQL-SYNTAX-IDENTIFIERS

This was causing some issues with metrics gathering in the past, but
that should be resolved now. This makes it more in-line with what
PostgreSQL is allowing and expecting. The max length is compile-time
configurable, as PostgreSQL technically provides a way to increase the
byte limit, by default we use current PostgreSQL default there.

This mean that some users *may* be allowed to use weird names, but
assuming that Victoria Metrics is properly parsing exported metrics
using OpenMetrics specs, then it should work there.

Close #579

1 of 1 new or added line in 1 file covered. (100.0%)

964 of 2066 relevant lines covered (46.66%)

203.75 hits per line