New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #8669 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

BUG: caches are bypassed when fetching distfiles

Reported by: josh_root@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: base Version: 1.2
Keywords: Cc:
Port:

Description

A "Pragma: no-cache" header is always sent when DP fetches distfiles via HTTP. There doesn't seem to be any good reason for this to be the default behaviour, it's just wasting bandwidth.

Attachments

curl.c.diff (1.2 KB) - added by blb@… 7 years ago.
Patch to base

Change History

Changed 7 years ago by blb@…

Patch to base

comment:1 Changed 7 years ago by jberry@…

I know I'm dense, but, a couple of questions...

So we use curl as a client. Whether or not we send a no-cache header, it shouldn't mean much of anything (unless there's a proxy involved?) since the server presumably doesn't respond to it. So this bug reports that we should change the default's used by curl because we use a few extra bytes in request headers over the wire?

Just making sure I'm clear..

comment:2 Changed 7 years ago by josh_root@…

No, I'm not worried about the header being sent per se. It's the result it causes with caching proxies, as you guessed. There's people (like me) who run Squid on their LAN, and lots of ISPs use a transparent proxy to cache HTTP traffic.

comment:3 Changed 7 years ago by josh_root@…

I tested the patch and it appears to have the desired effect.

comment:4 Changed 7 years ago by pguyot@…

  • Status changed from new to closed
  • Resolution set to fixed

Thanks. Committed.

Note: See TracTickets for help on using tickets.