when have "\" in regex, protoc will throw err" Invalid escape sequence" about go-proto-validators HOT 6 OPEN

\\ in .proto file generated valid regex in Go code. Maybe add that to documentation and close this issue?

from go-proto-validators.

I'm running into the same thing, but not using quotes gives me a different error:

pkg/proto/nks.proto:48:50: Error while parsing option value for "field": Expected string, got: /

Is there a fix for this?

from go-proto-validators.

Hello @chaoweili and @erstaples. I've only very recently picked up this repository for maintenance. I'll try to address this issue in the near term.

from go-proto-validators.

The quick fix in the mean time should be to specify your regex within back-ticks instead of double-quotes.

from go-proto-validators.

Thanks for the response, @Helcaraxan . That doesn't work for me, unfortunately. My compiler really wants those quotes:

builder_api/builder_api.proto:113:29: Error while parsing option value for "field": Expected string, got: `

Here's the field in question:

    string kubernetes_version = 8 [
        (validator.field) = { regex: `/^v[0-9]{1}\.[0-9]{1,2}\.[0-9]{1,2}$/` }
    ];

from go-proto-validators.

Reading your regular expression (RE) a few questions come to mind:

• Why the / at the front and the head of the RE? The ^ and $ already express the need to match an entire line?
• I believe that the following should work and give you the result you want:

    string kubernetes_version = 8 [
        (validator.field) = { regex: "^v[0-9]{1}\\.[0-9]{1,2}\\.[0-9]{1,2}$" }
    ];

Bear in mind that the string you provide in the regex validator is reused as is to instantiate a regexp.Regexp from the Go standard library so the RE should follow Go's RE syntax. Just wanted to make sure that was well understood, just in case. 🙂

from go-proto-validators.