プロジェクト

全般

プロフィール

Vote #68883

未完了

Caching of public pages, requested from anonymous users (10x - 20x speed improvements)

Admin Redmine さんが約2年前に追加.

ステータス:
New
優先度:
通常
担当者:
-
カテゴリ:
-
対象バージョン:
-
開始日:
2010/12/31
期日:
進捗率:

0%

予定工数:
category_id:
0
version_id:
0
issue_org_id:
7204
author_id:
25856
assigned_to_id:
0
comments:
12
status_id:
1
tracker_id:
3
plus1:
0
affected_version:
closed_on:
affected_version_id:
ステータス-->[New]

説明

I need a quicker access for public pages, requested from anonymous users.

I can not use a caching reverse proxy because the web proxy does not know when a page is requested from a registered user, or when the page is requested from an anonymous user.

In this patch the Ruby/Redmine application controller returns a page from the cache with ~ 10x - 20x speedup respecting normal pages. The caching is applied only if the user is anonymous (not logged) and only if the page does not contains forms with an authenticity token.

The cached page is different according the locale of user, so different pages for different locales are returned correctly.

The Redmine administrator can configure the cache refresh rate using the main Settings form.

The caching is performed using the caching method specified in the config/additional_environment.rb file. I added something like

config.action_controller.perform_caching = true
config.action_controller.cache_store = :file_store, File.join(File.dirname(__FILE__), '../tmp/cache')

I tested the patch on version 1.0.5, but it is not yet on a server in production.

The patch can be applied to r4595 or Git commit:6659c630b264325

Warnings / sorry in advance :-)

  • this is my first ruby code
  • this is my first ruby on rails code
  • I'm not a native english speaker, and probably my labels/texts is not completely correct

Many thanks for the good product that is Redmine!


journals

At least that setting:

<pre>config.action_controller.perform_caching = true</pre>

Is already part of the production environment. What environment and rails server do you use, and what kind of load are you having/expecting that this is a problem for you?
--------------------------------------------------------------------------------
Regarding caching, I had the same problem described in http://www.redmine.org/boards/2/topics/2432

I'm using nginx + phusion passenger + enterprise ruby on a local machine, and I have no public server up to date. I don't know in advance what will be my load.

--------------------------------------------------------------------------------
Massimo Zaniboni wrote:
> Regarding caching, I had the same problem described in http://www.redmine.org/boards/2/topics/2432

That's a wee bit old…

> I don't know in advance what will be my load.

Then you're probably trying to solve a problem you don't have.

Anyway, my experience is that the rails-provided facilities for caching are good enough (if configured correctly, of course), we don't get that much traffic over at https://orga.fachschaften.org/ , but the install is quite "snappy" nonetheless :-)
--------------------------------------------------------------------------------
I didn't found a lot of info about proper configuration of Redmine caching on the web.

I performed some test, without effective results. Then I studied Rails caching support, but I didn't found related instructions inside Redmine code, and my impression (but I'm no an expert), it is that Redmine uses very few Rails caching features.

--------------------------------------------------------------------------------
Redmine uses few caching technics indeed, only where it can improve response times significantly: activity rendering, diff rendering, textile rendering. This is generally not so easy to implement and we don't have much issues about performance reported here.

Concerning your patch, it looks like a good start and could be easily implemented as a plugin. I have a few comments:
* it seems reasonable to cache GET requests only
* how do you deal with parameters? Your cache key uses the request path only.
--------------------------------------------------------------------------------
> Concerning your patch, it looks like a good start and could be easily implemented as a plugin. I have a few comments:
> * it seems reasonable to cache GET requests only
> * how do you deal with parameters? Your cache key uses the request path only.

Good questions: add to the list that I don't know HTTP :-)

I focused better my problem: if Redmine pages will be cacheable, then http requests can be answered directly from a web proxy cache, that is 100 ~ 1000 faster than Redmine.

I will try to create a plugin, that when enabled:
* allows safe caching of pages for anonymous users, setting properly cache related http fields, removing unusued cookies, and so on;
* provide defaults pages for Issues, Roadmap and Activity pages, accessed from anonymous users, without customizable forms, but correctly cacheable;

In this way I can use Redmine both for project development (few registered users), and project presentation (hopefully many visitors), without any concerns about speed.

Is it a naive approach? Is it an already solved problem? Thanks in advance for any positive/negative feedback.

--------------------------------------------------------------------------------
Rails already does a lot of caching in production: page caching, fragment caching, and so on. I'd suggest you read up on those topics and look for the common or best practices regarding caching in rails before trying to reinvent the wheel.
--------------------------------------------------------------------------------
Massimo Zaniboni wrote:
> […] and project presentation (hopefully many visitors), without any concerns about speed.

Oh, and regarding that, I can only repeat myself: don't try to solve a problem you haven't run into or even approached in any significant way.

If you have numbers about expected traffic, do some tests with these numbers, try to find the bottlenecks and fix those, what you are trying to do in many ways replicates existing functionality (e.g. reverse proxies). If your only gripe with redmine is that the reverse proxy has no way to know if the user is anonymous, find a way to tell the proxy instead of trying to replicate the proxy in redmine.

Anyway, my 0,02€ on that topic.
--------------------------------------------------------------------------------
> Oh, and regarding that, I can only repeat myself: don't try to solve a problem you haven't run into or even approached in any significant way.

From a business point of view, you are completely right. But you should also consider that in the meantime I learn something about ruby on rails, web-proxy, and maybe I can release some work useful also to other people.

> If your only gripe with redmine is that the reverse proxy has no way to know if the user is anonymous, find a way to tell the proxy instead of trying to replicate the proxy in redmine.

Exactly the approach I described in my past message...

--------------------------------------------------------------------------------
Massimo Zaniboni wrote:
> > If your only gripe with redmine is that the reverse proxy has no way to know if the user is anonymous, find a way to tell the proxy instead of trying to replicate the proxy in redmine.
>
> Exactly the approach I described in my past message…

You're right, sorry, I was still thinking of your first approach.
--------------------------------------------------------------------------------
I created a plugin, for caching web-pages, using Varnish web cache, that completely substitute/improve my previous patch.

https://github.com/massimo-zaniboni/Redmine_cacheable_pages

It is not already tested in production.

This plugin works only applying the attached patch to Redmine. Maybe there is a better method for writing it, without modifying Redmine.

The Redmine forms will contain the authenticity_code, but it is a code associated to a public session exchanged between Varnish reverse web-proxy and Redmine server, and only for anonymous/not-logged users. I don't think this poses security problems. I were not able to remove sessions and authenticity code from Rails/Redmine.

I changed from POST to GET method inside ISSUES, when APPLY a query, in order to enable caching of ISSUE pages.

--------------------------------------------------------------------------------
Some clarifications, in order to facilitate inspecting of the code.

The logic of the plugin is inside

https://github.com/massimo-zaniboni/Redmine_cacheable_pages/blob/master/lib/redmine_cacheable_pages/hooks/cacheable_pages_hooks.rb

and

https://github.com/massimo-zaniboni/Redmine_cacheable_pages/blob/master/varnish/config_example.vcl

Because I were not able to remove Redmine session cookies from Redmine code (they are lazily produced whenever there is an access to a session field), I removed it on the Varnish side. It is not very elegant. Nginx configuration language was not powerful enough for coding all these transformations, so I used Varnish.

--------------------------------------------------------------------------------

表示するデータがありません

他の形式にエクスポート: Atom PDF

いいね!0