List for cgit developers and users
 help / color / mirror / Atom feed
From: valentin.haenel at gmx.de (Valentin Haenel)
Subject: [PATCHv4 0/2] Authorize viewing a repository
Date: Wed, 31 Oct 2012 19:50:40 +0100	[thread overview]
Message-ID: <1351709440-29185-1-git-send-email-valentin.haenel@gmx.de> (raw)
In-Reply-To: <1350378927-10834-1-git-send-email-valentin.haenel@gmx.de>

This is the fourth iteration of the patches to invoke an external helper for
authorizing viewing of repositories. The significant changes are that now, no
book-keeping of the REMOTE_USER or equivalent environment variable is done
within cgit and the authorization helper must instead obtain this from the
environment. Also, the authorization helper is no longer invoked via 'system',
but instead using a 'fork/exec/wait' construct. Lastly, the external helper is
no longer given any arguments, but must get the repository to authorize for,
from the environment.  To this end, the same "CGIT_*" environment variables
that are available to the filters are handed over. The cgit man page entry for
the authorization helper has been updated to describe the environment available
to the helper and the expected exit code.

The example for interfacing with gitolite has now got adequately quoted
variables and has been updated to reflect the changes in how the external
helper should be invoked.

Carlos Aguado Sanchez (1):
  Helper script to interface to gitolite

Valentin Haenel (1):
  Add ability to authorize viewing a repository

 cgit.c                |   46 ++++++++++++++++++++++++++++++++++++++++++++++
 cgit.h                |    1 +
 cgitrc.5.txt          |    6 ++++++
 contrib/gl-check-user |   20 ++++++++++++++++++++
 4 files changed, 73 insertions(+)
 create mode 100755 contrib/gl-check-user

-- 
1.7.9.5





  parent reply	other threads:[~2012-10-31 18:50 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-16  9:15 [PATCH 0/3] Implement authorization via external program valentin.haenel
2012-10-16  9:15 ` [PATCH 1/3] Add config option user-envvar valentin.haenel
2012-10-28  1:11   ` Jason
2012-10-29  9:49     ` valentin.haenel
2012-10-16  9:15 ` [PATCH 2/3] Add ability to authorize viewing a repository valentin.haenel
2012-10-16 11:21   ` Jason
2012-10-16 12:48     ` valentin.haenel
2012-10-16  9:15 ` [PATCH 2/3] Add ability to authorize viewing a repository using valentin.haenel
2012-10-16  9:17   ` valentin.haenel
2012-10-16  9:15 ` [PATCH 3/3] Helper script to interface to gitolite valentin.haenel
2012-10-22  8:29 ` [PATCHv2 1/3] Add config option user-envvar valentin.haenel
2012-10-28  1:00   ` mathstuf
2012-10-29  9:22     ` valentin.haenel
2012-10-29 14:53       ` mathstuf
2012-10-29 15:50         ` valentin.haenel
2012-10-22  8:29 ` [PATCHv2 2/3] Add ability to authorize viewing a repository valentin.haenel
2012-10-28  1:00   ` mathstuf
2012-10-28  1:16     ` Jason
2012-10-28  1:29       ` mathstuf
2012-10-28  1:33         ` Jason
2012-10-29  9:43       ` valentin.haenel
2012-10-29 14:52         ` mathstuf
2012-10-29 15:45           ` valentin.haenel
2012-10-28  1:17     ` Jason
2012-10-29 12:38       ` valentin.haenel
2012-10-30  9:54         ` valentin.haenel
2012-10-28  1:14   ` Jason
2012-10-29  9:36     ` valentin.haenel
2012-10-22  8:29 ` [PATCHv2 3/3] Helper script to interface to gitolite valentin.haenel
2012-10-28  1:00   ` mathstuf
2012-10-29  9:27     ` valentin.haenel
2012-10-30 10:11 ` [PATCHv3 0/3] Implement authorization via external program (v3) valentin.haenel
2012-10-30 15:04   ` mathstuf
2012-10-30 16:30   ` Jason
2012-10-30 10:11 ` [PATCHv3 1/3] Add config option user-envvar valentin.haenel
2012-10-30 16:29   ` Jason
2012-10-30 10:11 ` [PATCHv3 2/3] Add ability to authorize viewing a repository valentin.haenel
2012-10-30 16:30   ` Jason
2012-10-30 10:11 ` [PATCHv3 3/3] Helper script to interface to gitolite valentin.haenel
2012-10-30 15:05   ` mathstuf
2012-10-31 18:50 ` valentin.haenel [this message]
2012-10-31 18:52   ` [PATCHv4 1/2] Add ability to authorize viewing a repository valentin.haenel
2012-10-31 18:52   ` [PATCHv4 2/2] Helper script to interface to gitolite valentin.haenel
2012-11-01  3:03     ` jamie.couture
2012-11-01  3:23       ` mathstuf
2012-11-01  4:20         ` Jason
2012-11-01  4:31           ` mathstuf
2012-11-01  8:58           ` valentin.haenel
2012-11-01 17:32             ` Jason
2012-11-01 10:40   ` [PATCHv4 0/2] Authorize viewing a repository valentin.haenel
2012-11-01 17:27     ` Jason
2012-11-01 17:32       ` valentin.haenel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1351709440-29185-1-git-send-email-valentin.haenel@gmx.de \
    --to=cgit@lists.zx2c4.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).