author | Lars Hjemli <hjemli@gmail.com> | 2008-04-14 21:07:52 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-04-14 21:07:52 (UTC) |
commit | 78031f92b0ff064efa82065a3cf6020c1256a5fd (patch) (unidiff) | |
tree | 44c8d9925dc57149f22f15f1ec06ffd4c5c0a24e | |
parent | e9a7042b5b44c6af2c7dc91eabed732d92278218 (diff) | |
download | cgit-78031f92b0ff064efa82065a3cf6020c1256a5fd.zip cgit-78031f92b0ff064efa82065a3cf6020c1256a5fd.tar.gz cgit-78031f92b0ff064efa82065a3cf6020c1256a5fd.tar.bz2 |
Fix search in repo index even if caching is enabled
The repository index page needed to include the querystring in the cache
filename.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | cgit.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -134,97 +134,99 @@ static void querystring_cb(const char *name, const char *value) | |||
134 | ctx.qry.ofs = atoi(value); | 134 | ctx.qry.ofs = atoi(value); |
135 | } else if (!strcmp(name, "path")) { | 135 | } else if (!strcmp(name, "path")) { |
136 | ctx.qry.path = trim_end(value, '/'); | 136 | ctx.qry.path = trim_end(value, '/'); |
137 | } else if (!strcmp(name, "name")) { | 137 | } else if (!strcmp(name, "name")) { |
138 | ctx.qry.name = xstrdup(value); | 138 | ctx.qry.name = xstrdup(value); |
139 | } | 139 | } |
140 | } | 140 | } |
141 | 141 | ||
142 | static void prepare_context(struct cgit_context *ctx) | 142 | static void prepare_context(struct cgit_context *ctx) |
143 | { | 143 | { |
144 | memset(ctx, 0, sizeof(ctx)); | 144 | memset(ctx, 0, sizeof(ctx)); |
145 | ctx->cfg.agefile = "info/web/last-modified"; | 145 | ctx->cfg.agefile = "info/web/last-modified"; |
146 | ctx->cfg.cache_dynamic_ttl = 5; | 146 | ctx->cfg.cache_dynamic_ttl = 5; |
147 | ctx->cfg.cache_max_create_time = 5; | 147 | ctx->cfg.cache_max_create_time = 5; |
148 | ctx->cfg.cache_repo_ttl = 5; | 148 | ctx->cfg.cache_repo_ttl = 5; |
149 | ctx->cfg.cache_root = CGIT_CACHE_ROOT; | 149 | ctx->cfg.cache_root = CGIT_CACHE_ROOT; |
150 | ctx->cfg.cache_root_ttl = 5; | 150 | ctx->cfg.cache_root_ttl = 5; |
151 | ctx->cfg.cache_static_ttl = -1; | 151 | ctx->cfg.cache_static_ttl = -1; |
152 | ctx->cfg.css = "/cgit.css"; | 152 | ctx->cfg.css = "/cgit.css"; |
153 | ctx->cfg.logo = "/git-logo.png"; | 153 | ctx->cfg.logo = "/git-logo.png"; |
154 | ctx->cfg.max_commit_count = 50; | 154 | ctx->cfg.max_commit_count = 50; |
155 | ctx->cfg.max_lock_attempts = 5; | 155 | ctx->cfg.max_lock_attempts = 5; |
156 | ctx->cfg.max_msg_len = 60; | 156 | ctx->cfg.max_msg_len = 60; |
157 | ctx->cfg.max_repodesc_len = 60; | 157 | ctx->cfg.max_repodesc_len = 60; |
158 | ctx->cfg.module_link = "./?repo=%s&page=commit&id=%s"; | 158 | ctx->cfg.module_link = "./?repo=%s&page=commit&id=%s"; |
159 | ctx->cfg.renamelimit = -1; | 159 | ctx->cfg.renamelimit = -1; |
160 | ctx->cfg.robots = "index, nofollow"; | 160 | ctx->cfg.robots = "index, nofollow"; |
161 | ctx->cfg.root_title = "Git repository browser"; | 161 | ctx->cfg.root_title = "Git repository browser"; |
162 | ctx->cfg.script_name = CGIT_SCRIPT_NAME; | 162 | ctx->cfg.script_name = CGIT_SCRIPT_NAME; |
163 | ctx->page.mimetype = "text/html"; | 163 | ctx->page.mimetype = "text/html"; |
164 | ctx->page.charset = PAGE_ENCODING; | 164 | ctx->page.charset = PAGE_ENCODING; |
165 | ctx->page.filename = NULL; | 165 | ctx->page.filename = NULL; |
166 | } | 166 | } |
167 | 167 | ||
168 | static int cgit_prepare_cache(struct cacheitem *item) | 168 | static int cgit_prepare_cache(struct cacheitem *item) |
169 | { | 169 | { |
170 | if (!ctx.repo && ctx.qry.repo) { | 170 | if (!ctx.repo && ctx.qry.repo) { |
171 | ctx.page.title = fmt("%s - %s", ctx.cfg.root_title, | 171 | ctx.page.title = fmt("%s - %s", ctx.cfg.root_title, |
172 | "Bad request"); | 172 | "Bad request"); |
173 | cgit_print_http_headers(&ctx); | 173 | cgit_print_http_headers(&ctx); |
174 | cgit_print_docstart(&ctx); | 174 | cgit_print_docstart(&ctx); |
175 | cgit_print_pageheader(&ctx); | 175 | cgit_print_pageheader(&ctx); |
176 | cgit_print_error(fmt("Unknown repo: %s", ctx.qry.repo)); | 176 | cgit_print_error(fmt("Unknown repo: %s", ctx.qry.repo)); |
177 | cgit_print_docend(); | 177 | cgit_print_docend(); |
178 | return 0; | 178 | return 0; |
179 | } | 179 | } |
180 | 180 | ||
181 | if (!ctx.repo) { | 181 | if (!ctx.repo) { |
182 | item->name = xstrdup(fmt("%s/index.html", ctx.cfg.cache_root)); | 182 | item->name = xstrdup(fmt("%s/index.%s.html", |
183 | ctx.cfg.cache_root, | ||
184 | cache_safe_filename(ctx.qry.raw))); | ||
183 | item->ttl = ctx.cfg.cache_root_ttl; | 185 | item->ttl = ctx.cfg.cache_root_ttl; |
184 | return 1; | 186 | return 1; |
185 | } | 187 | } |
186 | 188 | ||
187 | if (!ctx.qry.page) { | 189 | if (!ctx.qry.page) { |
188 | item->name = xstrdup(fmt("%s/%s/index.%s.html", ctx.cfg.cache_root, | 190 | item->name = xstrdup(fmt("%s/%s/index.%s.html", ctx.cfg.cache_root, |
189 | cache_safe_filename(ctx.repo->url), | 191 | cache_safe_filename(ctx.repo->url), |
190 | cache_safe_filename(ctx.qry.raw))); | 192 | cache_safe_filename(ctx.qry.raw))); |
191 | item->ttl = ctx.cfg.cache_repo_ttl; | 193 | item->ttl = ctx.cfg.cache_repo_ttl; |
192 | } else { | 194 | } else { |
193 | item->name = xstrdup(fmt("%s/%s/%s/%s.html", ctx.cfg.cache_root, | 195 | item->name = xstrdup(fmt("%s/%s/%s/%s.html", ctx.cfg.cache_root, |
194 | cache_safe_filename(ctx.repo->url), | 196 | cache_safe_filename(ctx.repo->url), |
195 | ctx.qry.page, | 197 | ctx.qry.page, |
196 | cache_safe_filename(ctx.qry.raw))); | 198 | cache_safe_filename(ctx.qry.raw))); |
197 | if (ctx.qry.has_symref) | 199 | if (ctx.qry.has_symref) |
198 | item->ttl = ctx.cfg.cache_dynamic_ttl; | 200 | item->ttl = ctx.cfg.cache_dynamic_ttl; |
199 | else if (ctx.qry.has_sha1) | 201 | else if (ctx.qry.has_sha1) |
200 | item->ttl = ctx.cfg.cache_static_ttl; | 202 | item->ttl = ctx.cfg.cache_static_ttl; |
201 | else | 203 | else |
202 | item->ttl = ctx.cfg.cache_repo_ttl; | 204 | item->ttl = ctx.cfg.cache_repo_ttl; |
203 | } | 205 | } |
204 | return 1; | 206 | return 1; |
205 | } | 207 | } |
206 | 208 | ||
207 | struct refmatch { | 209 | struct refmatch { |
208 | char *req_ref; | 210 | char *req_ref; |
209 | char *first_ref; | 211 | char *first_ref; |
210 | int match; | 212 | int match; |
211 | }; | 213 | }; |
212 | 214 | ||
213 | int find_current_ref(const char *refname, const unsigned char *sha1, | 215 | int find_current_ref(const char *refname, const unsigned char *sha1, |
214 | int flags, void *cb_data) | 216 | int flags, void *cb_data) |
215 | { | 217 | { |
216 | struct refmatch *info; | 218 | struct refmatch *info; |
217 | 219 | ||
218 | info = (struct refmatch *)cb_data; | 220 | info = (struct refmatch *)cb_data; |
219 | if (!strcmp(refname, info->req_ref)) | 221 | if (!strcmp(refname, info->req_ref)) |
220 | info->match = 1; | 222 | info->match = 1; |
221 | if (!info->first_ref) | 223 | if (!info->first_ref) |
222 | info->first_ref = xstrdup(refname); | 224 | info->first_ref = xstrdup(refname); |
223 | return info->match; | 225 | return info->match; |
224 | } | 226 | } |
225 | 227 | ||
226 | char *find_default_branch(struct cgit_repo *repo) | 228 | char *find_default_branch(struct cgit_repo *repo) |
227 | { | 229 | { |
228 | struct refmatch info; | 230 | struct refmatch info; |
229 | 231 | ||
230 | info.req_ref = repo->defbranch; | 232 | info.req_ref = repo->defbranch; |