OK, thanks for he explanation. While agree about the part that using dynamic DH parameters might not provide you better security I am always amazed that some one at some point will actually point out that fixed parameters allow a security breach even as unlikely it appeared at the time the decision to use fixed parameters was made.So, could you just comment out the deleted code and make it obvious what was changed. I know this is being pedantic and somewhat paranoid but you know how security works. After that I have no problems with the patch subject to testing.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/2026#issuecomment-1578954551", "url": "https://github.com/xrootd/xrootd/pull/2026#issuecomment-1578954551", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1