clean up and move cert-related functions to new `cert.{c,h}`

siemens / libsecutils

Wrapper library for commonly used OpenSSL features, providing a simpler, more application-oriented API.

Other

5 stars 4 forks source link

clean up and move cert-related functions to new `cert.{c,h}` #12

Closed DDvO closed 2 years ago

DDvO commented 2 years ago

Also generalize UTIL_warn_cert{,s} to CERT_check{,_all}

DDvO commented 2 years ago

Benjamin, this PR looks bulky, but most of it just just shuffling code and header items to the new files where they fit much better.

DDvO commented 2 years ago

Whenever changing function names, e.g., UTIL_print_cert() to CERT_print(), I added a #define for API backward compatibility.

DDvO commented 2 years ago

@benjamin-schilling would it help reviewing if I split this PR into one renaming and adapting functions and one with the rest, i.e., moving contents from various places to cert.{c,h}?

DDvO commented 2 years ago

Merged to master, squashing fixups - thanks @benjamin-schilling for the approval.