ws4sqlite icon indicating copy to clipboard operation
ws4sqlite copied to clipboard

0.17 fork tries to read database file as config file

Open blaix opened this issue 9 months ago • 2 comments

I've mostly been using the 0.17 fork with --quick-db so didn't notice until now that when I use the --db option I get a strange error about control characters in the config file:

FATAL | 2025/03/16 22:29:09 | in parsing config file: yaml: control characters are not allowed

This happens even if the config file does not exist.

It looks like the problem might be that it never replaces the extension with yaml before getting here: https://github.com/proofrock/ws4sqlite/blob/5c04e3726c49af50553c5a73d9e0f9489738ebfb/src/cli.go#L93-L94

I built my own version with an added line right before that to print out yamlFile and when I run with -db test.db it prints test.db.

blaix avatar Mar 17 '25 02:03 blaix

The problem was that I was using 0.16 docs. The new behavior is to specify the path to the yaml file, not the database. So this is working correctly.

blaix avatar Mar 17 '25 10:03 blaix

I reopen this, as a reminder for at least a better wording of this error, at least; it should say that it's not YAML as expected.

proofrock avatar Mar 17 '25 11:03 proofrock