feat: make scram an optional feature

sunng87 / pgwire

PostgreSQL wire protocol implemented as a rust library.

Apache License 2.0

511 stars 36 forks source link

feat: make scram an optional feature #180

Closed sunng87 closed 5 months ago

sunng87 commented 5 months ago

This patch makes scram an optional feature completely as it introduces a few cypto libraries you might not want if you do not use scram in your application.

cc @serprex , this changes some behaviour introduced in #179

sunng87 commented 5 months ago

The updated feature sets:

server-api-aws-lc-rs as default, to align with upstream's defaults. It includes tokio-rustls with aws-lc-rs turned on
server-api-ring as alternative to use ring for rustls
scram as optional feature to enable scram authentication, it has to be used together with either server-api-aws-lc-rs or server-api-ring

serprex commented 5 months ago

x509-certificate brings in ring still so scram will always bring ring & aws-lc-rs unless using server-api-ring. But that's implementation detail, maybe in future x509-certificate can add aws-lc-rs feature. Otherwise LGTM