Pass BAO_ADDR to the token helper

[ruuda]

Previously the token helper might inherit BAO_ADDR from the process, but if the address was specified through an -address command-line flag, then the token helper would not know the address, or it would use the wrong one. Fix that by propagating the address everywhere, and then setting BAO_ADDR explicitly in the token helper's environment.

Resolves #314.

• One thing I’m not sure about: should we set VAULT_ADDR in addition to BAO_ADDR? I see there are still many uses of VAULT_ADDR in the codebase. ⇒ Yes, set both.

Target Release

It filled out this by default: 1.14.7

[cipherboy]

I'd copy from #321 and set VAULT_ADDR as well. :-)

[cipherboy]

\o hello @ruuda -- I think there's some test failures introduced by this PR. If I rebuild this PR on top of my #362, I think in command/login_test.go there's several places where cmd.TokenHelper() is called without the vaultAddr string.

Sorry for the noisy test failures, I hadn't realized some of my PRs had been merged without passing tests :o

ok  	github.com/openbao/openbao/builtin/logical/openldap/client	(cached)
# github.com/openbao/openbao/command [github.com/openbao/openbao/command.test]
command/login_test.go:65:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:118:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:164:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:205:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:241:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:299:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:359:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:420:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:494:23: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:545:22: not enough arguments in call to cmd.TokenHelper
	have ()
	want (string)
command/login_test.go:545:22: too many errors