Re: expires до конца сут ок

[Igor Sysoev <is@xxxxxxxxxxxxx>]

On Mon, Aug 11, 2008 at 06:08:03PM +0400, Igor Sysoev wrote:

> On Mon, Aug 11, 2008 at 03:44:07PM +0400, abs@xxxxxxxxxxxxxx wrote:
> 
> > Необходимо, чтобы в expires указывалось время оставшееся до конца суток. 
> > Как 
> > это можно сделать?
> > 
> > Хочется снизить нагрузку на сервер. На сервере несколько тысяч картинок, 
> > генерятся по запросу пользователей (естественно, кладутся на диск и 
> > раздаются 
> > потом) и остаются актуальными до конца суток. С началом новых суток старые 
> > удаляются, генерятся новые. Пользователь может их сгенерировать (при первом 
> > обращении) в любое время, поэтому  expires     modified +24h; не подходит.
> > По логам видно, что пользователь, получив первый раз картинку, в 
> > последствие 
> > просто дёргает сервер и получает код 304. И таких событий сотни тысяч. 
> > Указание expires до конца суток должно спасти отца русской демократии...
> 
> Прилагаемый патч позволяет задать время в виде:
> 
>      expires  @23h30m;
> 
> Время локальное.

Собственно патч.


-- 
Игорь Сысоев
http://sysoev.ru

Index: src/http/modules/ngx_http_headers_filter_module.c
===================================================================
--- src/http/modules/ngx_http_headers_filter_module.c   (revision 1479)
+++ src/http/modules/ngx_http_headers_filter_module.c   (working copy)
@@ -36,6 +36,7 @@
 #define NGX_HTTP_EXPIRES_MAX       2
 #define NGX_HTTP_EXPIRES_ACCESS    3
 #define NGX_HTTP_EXPIRES_MODIFIED  4
+#define NGX_HTTP_EXPIRES_DAILY     5
 
 
 typedef struct {
@@ -187,7 +188,7 @@
 ngx_http_set_expires(ngx_http_request_t *r, ngx_http_headers_conf_t *conf)
 {
     size_t            len;
-    time_t            since;
+    time_t            now, expires_time, max_age;
     ngx_uint_t        i;
     ngx_table_elt_t  *expires, *cc, **ccp;
 
@@ -279,16 +280,24 @@
         return NGX_OK;
     }
 
+    now = ngx_time();
+
     if (conf->expires == NGX_HTTP_EXPIRES_ACCESS
         || r->headers_out.last_modified_time == -1)
     {
-        since = ngx_time();
+        expires_time = now + conf->expires_time;
+        max_age = conf->expires_time;
 
+    } else if (conf->expires == NGX_HTTP_EXPIRES_DAILY) {
+        expires_time = ngx_next_time(conf->expires_time);
+        max_age = expires_time - now;
+
     } else {
-        since = r->headers_out.last_modified_time;
+        expires_time = r->headers_out.last_modified_time + conf->expires_time;
+        max_age = expires_time - now;
     }
 
-    ngx_http_time(expires->value.data, since + conf->expires_time);
+    ngx_http_time(expires->value.data, expires_time);
 
     if (conf->expires_time < 0) {
         cc->value.len = sizeof("no-cache") - 1;
@@ -303,8 +312,7 @@
         return NGX_ERROR;
     }
 
-    cc->value.len = ngx_sprintf(cc->value.data, "max-age=%T",
-                                since + conf->expires_time - ngx_time())
+    cc->value.len = ngx_sprintf(cc->value.data, "max-age=%T", max_age)
                     - cc->value.data;
 
     return NGX_OK;
@@ -514,11 +522,22 @@
         n = 2;
     }
 
-    if (value[n].data[0] == '+') {
+    if (value[n].data[0] == '@') {
         value[n].data++;
         value[n].len--;
         minus = 0;
 
+        if (hcf->expires == NGX_HTTP_EXPIRES_MODIFIED) {
+            return "daily time can not be used with \"modified\" parameter";
+        }
+
+        hcf->expires = NGX_HTTP_EXPIRES_DAILY;
+
+    } else if (value[n].data[0] == '+') {
+        value[n].data++;
+        value[n].len--;
+        minus = 0;
+
     } else if (value[n].data[0] == '-') {
         value[n].data++;
         value[n].len--;
Index: src/core/ngx_times.c
===================================================================
--- src/core/ngx_times.c        (revision 1479)
+++ src/core/ngx_times.c        (working copy)
@@ -291,3 +291,42 @@
     tp->ngx_tm_year = (ngx_tm_year_t) year;
     tp->ngx_tm_wday = (ngx_tm_wday_t) wday;
 }
+
+
+time_t
+ngx_next_time(time_t when)
+{
+    time_t     now, next;
+    struct tm  tm;
+
+    now = ngx_time();
+
+    ngx_libc_localtime(now, &tm);
+
+    tm.tm_hour = (int) (when / 3600);
+    when %= 3600;
+    tm.tm_min = (int) (when / 60);
+    tm.tm_sec = (int) (when % 60);
+
+    next = mktime(&tm);
+
+    if (next == -1) {
+        return -1;
+    }
+
+    if (next - now > 0) {
+        return next;
+    }
+
+    tm.tm_mday++;
+
+    /* mktime() should normalize a date (Jan 32, etc) */
+
+    next = mktime(&tm);
+
+    if (next != -1) {
+        return next;
+    }
+
+    return -1;
+}
Index: src/core/ngx_times.h
===================================================================
--- src/core/ngx_times.h        (revision 1479)
+++ src/core/ngx_times.h        (working copy)
@@ -25,7 +25,10 @@
 u_char *ngx_http_cookie_time(u_char *buf, time_t t);
 void ngx_gmtime(time_t t, ngx_tm_t *tp);
 
+time_t ngx_next_time(time_t when);
+#define ngx_next_time_n      "mktime()"
 
+
 extern volatile ngx_time_t  *ngx_cached_time;
 
 #define ngx_time()           ngx_cached_time->sec