Hello Thomas,
thank you for reviewing our patch.
> Why did you put "trust" there instead of "$authmethod" like the previous lines?
That is a good question in deed. We changed that accordingly.
> The tests pass and show the feature working correctly. I think this
> is getting close to committable.
We implemented all other suggestions in the attached patch version...
> I see that Magnus has signed up a committer.
... and changed status to "Ready for Committer" :-).
Kind regards,
Marius Timmer
--
Westfälische Wilhelms-Universität Münster (WWU)
Zentrum für Informationsverarbeitung (ZIV)
Röntgenstraße 7-13
Besucheradresse: Einsteinstraße 60 - Raum 101
48149 Münster
+49 251 83 31158
marius.timmer@uni-muenster.de
https://www.uni-muenster.de/ZIV