author | Lars Hjemli <hjemli@gmail.com> | 2009-08-10 07:20:17 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2009-08-10 07:20:17 (UTC) |
commit | 8b2252b0b61617e9de9d9e9ba743881ad62523af (patch) (side-by-side diff) | |
tree | 2d0f02d2c4a213644a1031a88af6578c032edbe6 | |
parent | 60a26272e0ca529407fe6b613f061f04ba585d53 (diff) | |
download | cgit-8b2252b0b61617e9de9d9e9ba743881ad62523af.zip cgit-8b2252b0b61617e9de9d9e9ba743881ad62523af.tar.gz cgit-8b2252b0b61617e9de9d9e9ba743881ad62523af.tar.bz2 |
ui-shared: add support for NO_HTTP=1/--nohttp
cgit_print_http_headers() used to do nothing if 'embedded' was
specified in cgitrc, but that was wrong - we never want to skip the
headers when invoked as a CGI app. Sadly, there's no easy way to
detect if we're invoked as a CGI app or if we're invoked by another
CGI app, so for the latter case cgit needs to be invoked with either
--nohttp on the command line or NO_HTTP=1 in the environment.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | cgit.c | 4 | ||||
-rw-r--r-- | cgit.h | 1 | ||||
-rw-r--r-- | ui-shared.c | 2 |
3 files changed, 6 insertions, 1 deletions
@@ -255,2 +255,3 @@ static void prepare_context(struct cgit_context *ctx) ctx->env.https = xstrdupn(getenv("HTTPS")); + ctx->env.no_http = xstrdupn(getenv("NO_HTTP")); ctx->env.path_info = xstrdupn(getenv("PATH_INFO")); @@ -447,2 +448,5 @@ static void cgit_parse_args(int argc, const char **argv) } + if (!strcmp(argv[i], "--nohttp")) { + ctx.env.no_http = "1"; + } if (!strncmp(argv[i], "--query=", 8)) { @@ -213,2 +213,3 @@ struct cgit_environment { char *https; + char *no_http; char *path_info; diff --git a/ui-shared.c b/ui-shared.c index 538ddda..4175bd8 100644 --- a/ui-shared.c +++ b/ui-shared.c @@ -457,3 +457,3 @@ void cgit_print_http_headers(struct cgit_context *ctx) { - if (ctx->cfg.embedded) + if (ctx->env.no_http && !strcmp(ctx->env.no_http, "1")) return; |