Re: [patch] Update to libfetch
From: Dag-Erling Smørgrav (des_at_des.no)
Date: 02/22/05
- Previous message: Philip Murray: "FreeBSD, Samba and OSX clients"
- In reply to: Kelly Yancey: "[patch] Update to libfetch"
- Next in thread: Dag-Erling Smørgrav: "Re: [patch] Update to libfetch"
- Reply: Dag-Erling Smørgrav: "Re: [patch] Update to libfetch"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
To: Kelly Yancey <kbyanc@posi.net> Date: Tue, 22 Feb 2005 11:02:57 +0100
Kelly Yancey <kbyanc@posi.net> writes:
> Attached is a patch to address concerns raised by Pawel Worach with
> regards to the recent change to set TCP_NOPUSH when sending HTTP
> requests from libfetch. The previous revision also introduced a call
> to shutdown(2) to close the write half of the socket in order to force
> the queued request to be sent. While this should be perfectly
> acceptable behavior for a TCP client, it appears that squid provides a
> configuration option to disallow half-closed clients (which Pawel is
> currently using). As such, after introducing the shutdown(2) call,
> fetch(1) can no longer fetch files via HTTP through such proxies.
> To address this issue, the attached patch replaces the call to
> shutdown(2) with some socket option fiddling (clearing TCP_NOPUSH and
> setting TCP_NODELAY) which does the same job of forcing the client to
> write the queued request to the network without closing the write half
> of the socket. This feels a bit hackish to me, but gets the job done.
> Anyway, I would appreciate any feedback. Thanks,
Doesn't simply reading from the socket or selecting it for reading
flush the output queue?
DES
-- Dag-Erling Smørgrav - des@des.no _______________________________________________ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-net To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"
- Previous message: Philip Murray: "FreeBSD, Samba and OSX clients"
- In reply to: Kelly Yancey: "[patch] Update to libfetch"
- Next in thread: Dag-Erling Smørgrav: "Re: [patch] Update to libfetch"
- Reply: Dag-Erling Smørgrav: "Re: [patch] Update to libfetch"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
