Skip to content

Conversation

@ldez
Copy link
Contributor

@ldez ldez commented Sep 14, 2017

Description

Add CLI configuration for Web backend authentication.

Digest *Digest
Forward *Forward
HeaderField string
Basic *Basic `description:"Basic authentication."`
Copy link
Member

@juliens juliens Sep 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

 --web.auth                     Authentication for dashboard and API          (default "true")
 --web.auth.basic           Basic authentication.                                     (default "true")
 --web.auth.digest          Digest authentication.                                   (default "true")
 --web.auth.forward        Forward authentication.                                (default "true")

default: true ? how can I set the users ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default:true: it's a flaeg bug see #2080

@ldez
Copy link
Contributor Author

ldez commented Sep 14, 2017

type Basic struct {
	Users     `mapstructure:","`
	UsersFile string
}

Users is a slice of string, and it's used as a compostion for Basic.

It's impossible to handle this in the cli without breaking the TOML.

@ldez ldez closed this Sep 14, 2017
@ldez ldez deleted the feature/cli-web-auth branch September 14, 2017 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants