const METADATA_MAX_LEN: usize = MAX_OVERHEAD;
-type AllClients = HashMap<ClientName, ClientHandles>;
+#[derive(Debug)]
+struct Global {
+ config: config::InstanceConfigGlobal,
+ all_clients: HashMap<ClientName, Client>,
+}
-struct ClientHandles {
+#[derive(Debug)]
+struct Client {
ic: Arc<InstanceConfig>,
web: tokio::sync::mpsc::Sender<WebRequest>,
}
boundary_finder: multipart::BoundaryFinder,
reply_to: tokio::sync::oneshot::Sender<WebResponse>,
warnings: Warnings,
+ conn: Arc<String>,
}
/// Reply from client task to hyper worker pool task
type WebResponseData = Vec<u8>;
#[throws(PacketError)]
-pub fn route_packet(packet: Box<[u8]>, daddr: IpAddr) {
- trace!("xxx discarding packet daddr={:?} len={}", daddr, packet.len());
+pub fn route_packet(conn: &str, link: &dyn Display,
+ packet: Box<[u8]>, daddr: IpAddr)
+{
+ // xxx
+ trace!("{} {} discarding packet daddr={:?} len={}",
+ conn, link, daddr, packet.len());
}
async fn handle(
conn: Arc<String>,
- all_clients: Arc<AllClients>,
+ global: Arc<Global>,
req: hyper::Request<hyper::Body>
) -> Result<hyper::Response<hyper::Body>, hyper::http::Error> {
if req.method() == Method::GET {
let hmac_got = &hmac_got[0..hmac_got_l];
let client_name = client;
- let client = all_clients.get(&client_name);
+ let client = global.all_clients.get(&client_name);
// We attempt to hide whether the client exists we don't try to
// hide the hash lookup computationgs, but we do try to hide the
// boundary, start, &comp.name, &client.ic);
let (reply_to, reply_recv) = tokio::sync::oneshot::channel();
- trace!("{} request xxx={}", &client.ic, initial.len());
+ trace!("{} {} request, Content-Length={}",
+ &conn, &client_name, length_hint);
let wreq = WebRequest {
initial,
initial_remaining,
boundary_finder: boundary_finder.into_owned(),
body,
warnings: mem::take(&mut warnings),
- reply_to
+ reply_to,
+ conn: conn.clone(),
};
client.web.try_send(wreq)
warnings = reply.warnings;
let data = reply.data?;
- if ! warnings.warnings.is_empty()
- && log::log_enabled!(log::Level::Trace) {
- trace!("{} {}: warnings {:?}", &conn, &client_name, &warnings.warnings);
+ if warnings.warnings.is_empty() {
+ trace!("{} {} responding, {}",
+ &conn, &client_name, data.len());
+ } else {
+ debug!("{} {} responding, {} warnings={:?}",
+ &conn, &client_name, data.len(),
+ &warnings.warnings);
}
let data = hyper::Body::from(data);
warnings: default(),
};
- dbg!(&response);
try_send_response(ret.reply_to, response);
}
let WebRequest {
initial, initial_remaining, length_hint, mut body,
boundary_finder,
- reply_to, mut warnings,
+ reply_to, conn, mut warnings,
} = req.ok_or_else(|| anyhow!("webservers all shut down!"))?;
match async {
)?
Ok::<_,AE>($ret)
})().context(stringify!($v))?;
- dbg!(&$v);
+ //dbg!(&$v);
}
}
if saddr != ic.link.client.0 { throw!(PE::Src(saddr)) }
let daddr = ip_packet_addr::<true>(header)?;
Ok(daddr)
- }, |(daddr,packet)| route_packet(daddr,packet),
- |e| Ok::<_,SlipFramesError<_>>({ warnings.add(&e)?; })
+ }, |(daddr,packet)| route_packet(
+ &conn, &ic.link.client, daddr,packet
+ ),
+ |e| Ok::<_,SlipFramesError<_>>({ warnings.add(&e)?; })
)?;
}
"hippotatd", LinkEnd::Server,
&opts.config, &opts.log, |ics|
{
- let global = config::InstanceConfigGlobal::from(&ics);
- let ipif = Ipif::start(&global.ipif, None)?;
+ let global_config = config::InstanceConfigGlobal::from(&ics);
- let all_clients: AllClients = ics.into_iter().map(|ic| {
- let ic = Arc::new(ic);
+ let ipif = Ipif::start(&global_config.ipif, None)?;
- let (web_send, web_recv) = mpsc::channel(
+ let ics = ics.into_iter().map(Arc::new).collect_vec();
+ let (client_handles_send, client_handles_recv) = ics.iter()
+ .map(|_ic| mpsc::channel(
5 // xxx should me max_requests_outstanding but that's
- // marked client-only so needs rework
- );
+ // marked client-only so needs rework
+ )).unzip::<_,_,Vec<_>,Vec<_>>();
+ let all_clients = izip!(
+ &ics,
+ client_handles_send,
+ ).map(|(ic, web_send)| {
+ (ic.link.client,
+ Client {
+ ic: ic.clone(),
+ web: web_send,
+ })
+ }).collect();
+
+ for (ic, web_recv) in izip!(
+ ics,
+ client_handles_recv,
+ ) {
let ic_ = ic.clone();
tasks.push((tokio::spawn(async move {
run_client(ic_, web_recv).await.void_unwrap_err()
}), format!("client {}", &ic)));
+ }
- (ic.link.client,
- ClientHandles {
- ic,
- web: web_send,
- })
- }).collect();
- let all_clients = Arc::new(all_clients);
-
- for addr in &global.addrs {
- let all_clients_ = all_clients.clone();
- let make_service = hyper::service::make_service_fn(move |conn| {
- let all_clients_ = all_clients_.clone();
- let conn = Arc::new(format!("[conn:{:?}]", conn));
- async { Ok::<_, Void>( hyper::service::service_fn(move |req| {
- handle(conn.clone(), all_clients_.clone(), req)
- }) ) } }
+ let global = Arc::new(Global {
+ config: global_config,
+ all_clients,
+ });
+
+ for addr in &global.config.addrs {
+ let global_ = global.clone();
+ let make_service = hyper::service::make_service_fn(
+ move |conn: &hyper::server::conn::AddrStream| {
+ let global_ = global_.clone();
+ let conn = Arc::new(format!("[{}]", conn.remote_addr()));
+ async { Ok::<_, Void>( hyper::service::service_fn(move |req| {
+ handle(conn.clone(), global_.clone(), req)
+ }) ) }
+ }
);
- let addr = SocketAddr::new(*addr, global.port);
+ let addr = SocketAddr::new(*addr, global.config.port);
let server = hyper::Server::try_bind(&addr)
.context("bind")?
.http1_preserve_header_case(true)
~ian / hippotat.git / blobdiff