Remove ServerBuilder::configure (#349)

actix-server/CHANGES.md

@@ -1,6 +1,10 @@
 # Changes
 
 ## Unreleased - 2021-xx-xx
+* Remove `config` module. `ServiceConfig`, `ServiceRuntime` public types are removed due to this change. [#349]
+* Remove `ServerBuilder::configure` [#349]
+
+[#349]: https://github.com/actix/actix-net/pull/349
 
 
 ## 2.0.0-beta.5 - 2021-04-20

actix-server/src/builder.rs

@@ -12,7 +12,6 @@ use tokio::sync::mpsc::{unbounded_channel, UnboundedReceiver};
 use tokio::sync::oneshot;
 
 use crate::accept::AcceptLoop;
-use crate::config::{ConfiguredService, ServiceConfig};
 use crate::server::{Server, ServerCommand};
 use crate::service::{InternalServiceFactory, ServiceFactory, StreamNewService};
 use crate::signals::{Signal, Signals};
@@ -149,32 +148,6 @@ impl ServerBuilder {
  self
  }
 
- /// Execute external configuration as part of the server building process.
- ///
- /// This function is useful for moving parts of configuration to a different module or
- /// even library.
- pub fn configure<F>(mut self, f: F) -> io::Result<ServerBuilder>
- where
- F: Fn(&mut ServiceConfig) -> io::Result<()>,
- {
- let mut cfg = ServiceConfig::new(self.threads, self.backlog);
-
- f(&mut cfg)?;
-
- if let Some(apply) = cfg.apply {
- let mut srv = ConfiguredService::new(apply);
- for (name, lst) in cfg.services {
- let token = self.token.next();
- srv.stream(token, name.clone(), lst.local_addr()?);
- self.sockets.push((token, name, MioListener::Tcp(lst)));
- }
- self.services.push(Box::new(srv));
- }
- self.threads = cfg.threads;
-
- Ok(self)
- }
-
  /// Add new service to the server.
  pub fn bind<F, U, N: AsRef<str>>(mut self, name: N, addr: U, factory: F) -> io::Result<Self>
  where

actix-server/src/lib.rs

@@ -6,7 +6,6 @@
 
 mod accept;
 mod builder;
-mod config;
 mod server;
 mod service;
 mod signals;
@@ -16,7 +15,6 @@ mod waker_queue;
 mod worker;
 
 pub use self::builder::ServerBuilder;
-pub use self::config::{ServiceConfig, ServiceRuntime};
 pub use self::server::Server;
 pub use self::service::ServiceFactory;
 pub use self::test_server::TestServer;

actix-server/src/service.rs

@@ -24,7 +24,7 @@ pub(crate) trait InternalServiceFactory: Send {
 
  fn clone_factory(&self) -> Box<dyn InternalServiceFactory>;
 
- fn create(&self) -> LocalBoxFuture<'static, Result<Vec<(Token, BoxedServerService)>, ()>>;
+ fn create(&self) -> LocalBoxFuture<'static, Result<(Token, BoxedServerService), ()>>;
 }
 
 pub(crate) type BoxedServerService = Box<
@@ -131,14 +131,14 @@ where
  })
  }
 
- fn create(&self) -> LocalBoxFuture<'static, Result<Vec<(Token, BoxedServerService)>, ()>> {
+ fn create(&self) -> LocalBoxFuture<'static, Result<(Token, BoxedServerService), ()>> {
  let token = self.token;
  let fut = self.inner.create().new_service(());
  Box::pin(async move {
  match fut.await {
  Ok(inner) => {
  let service = Box::new(StreamService::new(inner)) as _;
- Ok(vec![(token, service)])
+ Ok((token, service))
  }
  Err(_) => Err(()),
  }