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

Can't init database in a path having blanks #26

Closed
vvoody opened this issue Apr 14, 2016 · 5 comments
Closed

Can't init database in a path having blanks #26

vvoody opened this issue Apr 14, 2016 · 5 comments
Labels

Comments

@vvoody
Copy link

vvoody commented Apr 14, 2016

$ mkdir '/tmp/path having blanks/'
$ cd '/tmp/path having blanks/'
$ pwd
/tmp/path having blanks
$ go-cve-dictionary server
[Apr 14 10:49:25] ERROR SQLite3 DB path must be a *Absolute* file path. dbpath: /tmp/path having blanks/cve.sqlite3

If I run go-cve-dictionary server in a path without blanks in the name, that's fine.

BTW, could you approve my request to join slack team? I applied many days ago :(

@kotakanbe
Copy link
Member

@vvoody

BTW, could you approve my request to join slack team? I applied many days ago :(

Vuls Slack team is open team.
You will receive the confirm mail after submitting your email-address.

Please try again.

@kotakanbe
Copy link
Member

Blanks in the path

I will change the error message. Thanks!

@kotakanbe kotakanbe added the bug label Apr 14, 2016
@kotakanbe
Copy link
Member

Or you can send a pull request if you like! :-)

@vvoody
Copy link
Author

vvoody commented Apr 14, 2016

Oh, I missed the email :( Now, you saw me :)

And for this issue, I found it's caused by package govalidator's regex patterns, WinPath and UnixPath.
The patterns don't allow blanks. Opened an issue for them, asaskevich/govalidator#124.

WinPath        string = `^[a-zA-Z]:\\(?:[^\\/:*?"<>|\r\n]+\\)*[^\\/:*?"<>|\r\n]*$`
UnixPath       string = `^((?:\/[a-zA-Z0-9\.\:]+(?:_[a-zA-Z0-9\:\.]+)*(?:\-[\:a-zA-Z0-9\.]+)*)+\/?)$`

@kotakanbe
Copy link
Member

kotakanbe commented Apr 17, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants