-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
be18cc9
to
abd2d7e
Compare
@refs the flags obviously explode but I am not sure if I can split the server.go in multiple files. I am currently using only a single debug configuraton ... hm but moving to multiple flagset files would make that easier as well ... I'll try that tomorrow. |
to test
That should give you a working reva & phoenix try using an ldap server using env vars: |
19f16d5
to
920160b
Compare
basic auth works, checked
|
4f18404
to
3049767
Compare
update config add desktop client with all localhost ports split flagsets add user docs update readme Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
3049767
to
25a68c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ouch, was in the middle of a review when it got merged. I'll propose a way of dealing with reva config that is type safe and doesn't bloat this part of the codebase
|
||
The initial version would only allow us to use a set of reva configurations to start multiple services. | ||
We use a more opinionated set of commands to start dedicated services that allows us to configure them individually. | ||
It allowcs us to switch eg. the user backend to LDAP and fully it on the cli. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
allows*
the user backend to LDAP and fully it on the cli -> the user backend to LDAP fully it on the cli
fixes #6