search

MHumm / DelphiEncryptionCompendium

Cryptographic library for Embarcadero Delphi and potentially for FPC as well
Apache License 2.0
249 stars 65 forks source link

Idea: Mark broken ciphers as deprecated #36

Closed Gloegg closed 2 years ago

Gloegg commented 2 years ago

Maybe it's a good idea to mark ciphers, that shouldn't be used anymore as deprecated, so developers using these ciphers are warned when they use them.

MHumm commented 2 years ago

Maybe, but it has one consequence for me: I get deprecated warning all over in the unit tests. And the other issue is: who decides when a cipher is broken? Sometimes it's debatable or I sometimes cannot properly interpret the data telling how many iterations it takes to break it and if that's considered to be too week now... The manual (did you read it? ;-) ) contains a list though...

Gloegg commented 2 years ago

Yeah ok, that is true. I had this idea, when I saw the list in the manual and askey myself if this info couldn't be brought directly to the source code :-)

MHumm commented 2 years ago

While the idea is not bad I wouldn't like to clutter compilation with deprecated warnings for those users which are forced for some reasons to use old algorithms. So I close this as won't do. In addition many of those contain warnings not only in the documentation but in the XMLDOC as well.