feat: update pgwire to 0.34 and improve how we disable ssl (#3432)

# Description of Changes

Hello team, I'm the original author of pgwire library. In this patch, I
have some updates about spacetimedb's usage of pgwire library:

- Updated pgwire library to 0.34, which includes a critical fix for busy
loop on accepting new connection
- This version will return error on no-op handlers, as requested by
@mamcx
- Update `StartupHandler` for `SslRequest`. `SslRequest` is handled in
`process_socket` automatically before `StartupHandler` kicks in. So it's
safe to remove the match branch. The framework will reject TLS
negotiation automatically if no TLS support compiled
- Updated feature flags. Add `no-default-features` so we will not pull
TLS dependencies.

# API and ABI breaking changes

N/A

# Expected complexity level and risk

1

# Testing

Would be nice to verify if you have integration test on postgres
interface.

---------

Co-authored-by: Mario Montoya <[email protected]>

Author: sunng87

Cargo.lock

@@ -219,7 +219,7 @@ paste = "1.0"
 percent-encoding = "2.3"
 petgraph = { version = "0.6.5", default-features = false }
 pin-project-lite = "0.2.9"
-pgwire = { version = "0.32", features = ["server-api"] }
+pgwire = { version = "0.34.1", default-features = false, features = ["server-api", "pg-ext-types"] }
 postgres-types = "0.2.5"
 pretty_assertions = { version = "1.4", features = ["unstable"] }
 proc-macro2 = "1.0"

Cargo.toml

@@ -46,8 +46,6 @@ pub(crate) enum PgError {
     DatabaseNameRequired,
     #[error(transparent)]
     Pg(#[from] PgWireError),
-    #[error("SSL is not supported by SpacetimeDB")]
-    SSLNotSupported,
     #[error(transparent)]
     Other(#[from] anyhow::Error),
 }
@@ -150,7 +148,7 @@ struct PgSpacetimeDB<T> {
 }
 
 impl<T: ControlStateReadAccess + ControlStateWriteAccess + NodeDelegate + Clone> PgSpacetimeDB<T> {
-    async fn exe_sql<'a>(&self, query: String) -> PgWireResult<Vec<Response<'a>>> {
+    async fn exe_sql(&self, query: String) -> PgWireResult<Vec<Response>> {
         let params = self.cached.lock().await.clone().unwrap();
         let db = SqlParams {
             name_or_identity: database::NameOrIdentity::Name(DatabaseName(params.database.clone())),
@@ -282,12 +280,6 @@ impl<T: Sync + Send + ControlStateReadAccess + ControlStateWriteAccess + NodeDel
                 self.cached.lock().await.clone_from(&Some(metadata));
                 finish_authentication(client, &self.parameter_provider).await?;
             }
-            PgWireFrontendMessage::SslRequest(_) => {
-                let err = PgError::SSLNotSupported;
-                log::error!("{err}");
-                let err = ErrorInfo::new("FATAL".to_owned(), "28P01".to_owned(), err.to_string());
-                return close_client(client, err).await;
-            }
             // The other messages are for features not supported by SpacetimeDB, that are rejected by the parser.
             _ => {
                 unreachable!("Unsupported startup message: {message:?}");
@@ -301,7 +293,7 @@ impl<T: Sync + Send + ControlStateReadAccess + ControlStateWriteAccess + NodeDel
 impl<T: Sync + Send + ControlStateReadAccess + ControlStateWriteAccess + NodeDelegate + Clone> SimpleQueryHandler
     for PgSpacetimeDB<T>
 {
-    async fn do_query<'a, C>(&self, _client: &mut C, query: &str) -> PgWireResult<Vec<Response<'a>>>
+    async fn do_query<C>(&self, _client: &mut C, query: &str) -> PgWireResult<Vec<Response>>
     where
         C: ClientInfo + Unpin + Send + Sync,
     {