thegushi commented on PR #26: URL: https://github.com/apache/spamassassin/pull/26#issuecomment-4195056117
> > I have looked at the changes and they look good. I particularly like the idea of adding a debug option exposed to the command line. > > I wonder if the debug option should be a more generic option rather then dedicated to ssl debugging. As mentioned in [either some back-and-forth with KAM (who wrote the original SSL support) and giovanni, and/or the tickets/notes here], the debugs were mainly added to try and figure out the actual SSL issues I was debugging, since at that point you can't easily debug it with wireshark (encrypted), and a bunch of opaque tls-encoded strings in a debugger are....bad, especially since it requires a build with debug symbols, which most users in the field reporting an issue won't have. (In spamd, you'd just stick a couple of warn's in and be done with it, but since spamc requires a recompile...yeah, annoying). If you can think of other places we'd like to add debugs, I'll happily add them, either to a different -Option or just make them spit out with the SSL debugs. I'd argue that since this push was related specifically to ssl, it makes sense to not have this feature a full debug-featureset insert. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
