Re: Add regular expression testing for user name mapping in the peer authentication TAP test

2022-10-16 Thread Drouvot, Bertrand
Hi, On 10/17/22 4:07 AM, Michael Paquier wrote: On Sat, Oct 15, 2022 at 07:54:30AM +0200, Drouvot, Bertrand wrote: Right. Giving a second thought to the non matching case, I think I'd prefer to concatenate the system_user to the system_user instead. This is what v2 does. Fine by me, so applie

Re: Add regular expression testing for user name mapping in the peer authentication TAP test

2022-10-16 Thread Michael Paquier
On Sat, Oct 15, 2022 at 07:54:30AM +0200, Drouvot, Bertrand wrote: > Right. Giving a second thought to the non matching case, I think I'd prefer > to concatenate the system_user to the system_user instead. This is what v2 > does. Fine by me, so applied v2. Thanks! -- Michael signature.asc Descr

Re: Add regular expression testing for user name mapping in the peer authentication TAP test

2022-10-14 Thread Drouvot, Bertrand
Hi, On 10/15/22 5:11 AM, Michael Paquier wrote: On Fri, Oct 14, 2022 at 06:31:15PM +0200, Drouvot, Bertrand wrote: while working on [1], I thought it could also be useful to add regular expression testing for user name mapping in the peer authentication TAP test. Good idea now that we have a

Re: Add regular expression testing for user name mapping in the peer authentication TAP test

2022-10-14 Thread Michael Paquier
On Fri, Oct 14, 2022 at 06:31:15PM +0200, Drouvot, Bertrand wrote: > while working on [1], I thought it could also be useful to add regular > expression testing for user name mapping in the peer authentication TAP > test. Good idea now that we have a bit more coverage in the authenticat

Add regular expression testing for user name mapping in the peer authentication TAP test

2022-10-14 Thread Drouvot, Bertrand
Hi hackers, while working on [1], I thought it could also be useful to add regular expression testing for user name mapping in the peer authentication TAP test. This kind of test already exists in kerberos/t/001_auth.pl but the proposed one in the peer authentication testing would probably