Re: [PATCH] Enable icon caching for pkgdb

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 15, 2010 at 10:13:17AM -0500, Mike McGrath wrote:
> Toshio and I worked through this yesterday and agreed the change was worth
> it.  It's very easy to revert.  Can I get 2 +1's?
> 
> diff --git a/manifests/servergroups/proxy.pp
> b/manifests/servergroups/proxy.pp
> index 43e0c69..099305c 100644
> --- a/manifests/servergroups/proxy.pp
> +++ b/manifests/servergroups/proxy.pp
> @@ -401,7 +401,7 @@ class proxy {
>      fedora-packagedb::proxy { "admin.fedoraproject.org/pkgdb":
>          website  => "admin.fedoraproject.org",
>          path     => "/pkgdb",
> -        proxyurl => "http://localhost:10003";,
> +        proxyurl => "http://localhost:6081";,
>      }
> 
>      bodhi::proxy { "admin.fedoraproject.org/updates":
> diff --git a/modules/varnish/files/proxy.vcl
> b/modules/varnish/files/proxy.vcl
> index 2ba7414..327e1be 100644
> --- a/modules/varnish/files/proxy.vcl
> +++ b/modules/varnish/files/proxy.vcl
> @@ -112,8 +112,9 @@ sub vcl_recv {
>      if (req.url ~ "^/w/") {
>          set req.backend = wiki;
>      }
> -    if (req.url ~ "^/pkgdb/") {
> +    if (req.url ~ "^/pkgdb/appicon/show/") {
>          set req.backend = pkgdb;
> +        unset req.http.cookie;
>      }
>      if (req.url ~ "^/mirrorlist/") {
>          set req.backend = mirrorlists;
> @@ -187,6 +188,13 @@ sub vcl_recv {
>      lookup;
>  }
> 
> +# When requesting application icons, don't allow cherrypy to set cookies
> +sub vcl_fetch {
> +    if (req.url ~ "^/pkgdb/appicon/show/") {
> +        unset obj.http.set-cookie;
> +    }
> +}
> +
>  # Called if the cache has a copy of the page.
>  #sub vcl_hit {
>  #    if (req.request == "PURGE")
> 
> 
+1

-Toshio

Attachment: pgphzWTg5uT6I.pgp
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux