Ambiguous error message on required about configor

[lukasaron]

The error is correct I would recommend to change the name of the field from URL into something more specific. The error will be more helpful then.

[prochac]

Ok, but it's anoying to have to use

config.InputService.InputServiceURL

instead of

config.InputService.URL

I planed to do a pull request for this issue but than I realized that even full name is not what I want.

InputService.URL is required, but blank

What I expect is more like

IN_SERVICE_URL variable is required, but blank

So I'm not using this lib anymore :) Not because it's bad.
But because anything simpler, for environment variables only, is enough for me.

[lukasaron]

You are making assumptions based on the env variable in the tag. Such variable might be also empty when this value is filled from other source than ENV variable.

Existing logic returns the name of the field which is required and its value is empty (not defined in any source).

Based on your code I would suggest naming the field: InputURL or InURL and same format for output to distinguish between them.

[prochac]

Yeah, no doubt.

As I sayed, I don't use this lib anymore.

Less is more and it works for me.
I need to have the env vars only, following https://12factor.net/config, so I made a switch.
https://play.golang.org/p/ABZ7CuywNKQ
or good ol' os.Getenv and os.LookupEnv.

And instead of .env, yaml, toml, ... files you can use shell file for sourcing the dev environment.
And without adding the extra complexity to app.
$ . ./dev-env.sh
simple as pie