question - why must URI be filled in for config file to be accepted?

[GuitarBilly]

hi @deajan, welcome in the new year ;-)
As the title says, i wanted to have a 'skeleton' config file (to modify another existing config file later by text editor).

I selected "Create New Config" and got to the config screen.
However it refuses to Accept because I have to fill in an URI:

Why is this?

• I can leave all "Backup" tab items empty so only an URI does not mean a useful or valid backup will be done.
• the URI is not checked, i can type anything in the field and it is accepted.
• for the other 1000 things that can be forgotten or wronly input by the user, npbackup will throw errors and warnings.

( As for my personal opinion i'd rather keep things simple and consistent and remove this check )

[deajan]

Most settings can indeed be left alone, but Repo URI / path should never be empty, the same should apply for repo password.
Backup items can be too much different settings to test them all, but indeed should also be enforced.

I won't remove the test assertion, but I could change it into a warning, would that fit for you ?

[GuitarBilly]

@deajan yes a warning would work for me.
You are aware that the repo password can be left empty and is not checked at the moment?

Back to the Repo UTI / path, you check if the entry is not empty but you do not check if it is a valid repo/path.
And some strange things can happen:

1. create new config with only bladiebla as Repo URI/path, empty password.
2. Accept config
3. watch main window showing "no snapshots¨. no error or warning given
4. press launch backup
5. backup HANGS FOREVER with some errors : password cannot be empty (from restic) and Runner: cannot execute, backend not ready. Runner: Function backup failed with "NoneType' object has not attribute ´backup_result_content´.
6. pressing Cancel then Yes gives "unknown error please check logs. NoneType object has no attribute 'cancel'.
7. npbackup closes (crashed?)

sorry I do not have time to put screenshots but this should be easy to reproduce.

[Alk2]

i wanted to have a 'skeleton' config file (to modify another existing config file later by text editor).

@GuitarBilly Just a suggestion; could you use the Example Config files from the Github repository https://github.com/netinvent/npbackup/tree/main/examples to do what you need?

[GuitarBilly]

@Alk2 thanks as I cannot remember knowing these existed. I'd say yes definitely a valid and complete 'skeleton' and sufficient for what I had planned to do.

[Alk2]

@Alk2 thanks as I cannot remember knowing these existed. I'd say yes definitely a valid and complete 'skeleton' and sufficient for what I had planned to do.

Great! Just note that if on Windows, this little Bug that I discovered with the example file: #178

[deajan]

There is currently a major GUI rewrite going on, I'll take a look into the above before release.