jdoucerain

Results 11 comments of jdoucerain

@jpillora could you please have a look at the bit improved version of the password validation ? Thanks

@jpillora thanks for the review you did and I tried to follow what you said. Could you please review this change again ? Thanks again

Hi @jpillora would you have some time to check if you could merge this branch ?

Hi @jpillora could you please check if you could merge this branch ? Thanks

@jpillora please go ahead if you have some bandwidth. Thanks

> looking at the is PR now. it appears you still have to define users in the --auth or --authfile, and the "local" password still works. ldap just adds an...

Yes this is intentional. We have to cover case where the users could or could not be in the enterprise directory.

And the presence in the directory is not enough for us to allow anyone to use chisel. This is why we are maintaining the authfile for each individual. This is...

I think using ldap as authentication only and applying a default parameterized remote is another use case to be developed. And limiting the authentication to ldap only and blocking the...

Yes I agree with option 2 I will update the PR consequently