On 09/10/18 19:58, Rémy Maucherat wrote: > On Tue, Oct 9, 2018 at 8:56 PM Mark Thomas <ma...@apache.org> wrote: > >> On 09/10/18 19:55, ma...@apache.org wrote: >>> Author: markt >>> Date: Tue Oct 9 18:55:17 2018 >>> New Revision: 1843328 >>> >>> URL: http://svn.apache.org/viewvc?rev=1843328&view=rev >>> Log: >>> Fix unused code warning >> >> My IDE popped up a warning that this parameter was unused. I think I >> have fixed this correctly but additional review would be good here. >> > > Good idea. My own IDE never cares about anything.
Some further testing suggests my change wasn't what was intended (since there is a hard-coded default of "RSA" set further up the stack and that is an invalid value here). Was was intended? I can't see anywhere else where clientKeyAlgo could be used. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org