Return-Path: X-Original-To: apmail-incubator-bloodhound-commits-archive@minotaur.apache.org Delivered-To: apmail-incubator-bloodhound-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 546E0ECB5 for ; Wed, 2 Jan 2013 09:35:57 +0000 (UTC) Received: (qmail 86353 invoked by uid 500); 2 Jan 2013 09:35:57 -0000 Delivered-To: apmail-incubator-bloodhound-commits-archive@incubator.apache.org Received: (qmail 86309 invoked by uid 500); 2 Jan 2013 09:35:56 -0000 Mailing-List: contact bloodhound-commits-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: bloodhound-dev@incubator.apache.org Delivered-To: mailing list bloodhound-commits@incubator.apache.org Received: (qmail 86295 invoked by uid 99); 2 Jan 2013 09:35:55 -0000 Received: from bloodhound-vm.apache.org (HELO bloodhound-vm) (140.211.11.32) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 02 Jan 2013 09:35:55 +0000 Received: from bloodhound-vm.apache.org (localhost [127.0.0.1]) by bloodhound-vm (Postfix) with ESMTP id 9FF8B8123F; Wed, 2 Jan 2013 09:35:55 +0000 (UTC) MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit From: "Apache Bloodhound" X-Trac-Version: 1.0 Cc: bloodhound-commits@incubator.apache.org Auto-Submitted: auto-generated X-Mailer: Trac 1.0, by Edgewall Software X-Trac-Project: Apache Bloodhound Date: Wed, 02 Jan 2013 09:35:55 -0000 Reply-To: bloodhound-dev@incubator.apache.org X-URL: https://issues.apache.org/bloodhound/ Subject: Re: [Apache Bloodhound] #330: Lazy evaluation of req.chrome X-Trac-Ticket-URL: https://issues.apache.org/bloodhound/ticket/330#comment:1 Message-ID: <069.b6e088996409333013a34cc0911eef17@incubator.apache.org> References: <054.a8c49d5edac92b71161cd09f61c642a0@incubator.apache.org> X-Trac-Ticket-ID: 330 In-Reply-To: <054.a8c49d5edac92b71161cd09f61c642a0@incubator.apache.org> #330: Lazy evaluation of req.chrome --------------------------+-------------------- Reporter: peter | Owner: nobody Type: enhancement | Status: new Priority: minor | Milestone: Component: dashboard | Version: Resolution: | Keywords: --------------------------+-------------------- Changes (by peter): * priority: major => minor Old description: New description: For optimal performance req.chrome should be evaluated only if necessary. This currently violated on multiple places in request handlers, filters and mainly by TracThemeEngine plugin. BH's request handlers and filters could be fixed within BH (whitelabeling solution was already dicussed on dev list). However there has been no investigation on TracThemeEngine yet. Until this is fixed there is no benefits in fixing BH code at all. -- -- Ticket URL: Apache Bloodhound The Apache Bloodhound (incubating) issue tracker