Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev #13

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Dev #13

wants to merge 9 commits into from

Conversation

dietmarb01
Copy link

Hi Gavin,

thanks for this wonderful piece of software which we've been using for quite a few years now.

With some pressure from Ubuntu 14 LTS to move to Apache 2.4, I had to adapt it to run with it. In doing so, I also enabled it to benefit from the new authorization provider API.

Along the way, I also made a small correction to inconsistent char/char* use of the query_separator which caused Apache to sometimes core dump.

Greetings from Austria to Australia,
Dietmar

@dune73
Copy link

dune73 commented Apr 8, 2015

That's most welcome. Thank you Dietmar.

@gavincarr
Copy link
Owner

Hi Dietmar,

That looks excellent, thanks for your work. I'll try and and review it
in the next couple of weeks.

Cheers,
Gavin

On Tue, Apr 07, 2015 at 11:29:10AM -0700, Dietmar Berg wrote:

Hi Gavin,

thanks for this wonderful piece of software which we've been using for quite a few years now.

With some pressure from Ubuntu 14 LTS to move to Apache 2.4, I had to adapt it to run with it. In doing so, I also enabled it to benefit from the new authorization provider API.

Along the way, I also made a small correction to inconsistent char/char* use of the query_separator which caused Apache to sometimes core dump.

Greetings from Austria to Australia,
Dietmar
You can view, comment on, or merge this pull request online at:

#13

-- Commit Summary --

  • Minimum modifs to make it run on Apache 2.4 (Ubuntu LTS 14)
  • Minimum change for Apache 2.4, now based on latest GitHub version
  • Implements Apache 2.4 authz provider ("Require tkt-group group1 group2 ...")
  • Cleanup (extra blanks at end of line)
  • Updated/added doc for use with Apache 2.4; made TKTAuthToken deprecated in favor of Require
  • Merge branch 'feature/apache24' into dev
  • Updated manpage; less pressure to move away from 'TKTAuthToken' to 'Require'.
  • Apache 2.4 port of mod_auth_tkt with small enhancements.
  • Merge tag '2.2.0' into dev

-- File Changes --

M INSTALL (2)
A README.Apache_2.4 (45)
M configure (14)
M doc/mod_auth_tkt.pod (40)
M src/mod_auth_tkt.c (204)

-- Patch Links --

https://github.com/gavincarr/mod_auth_tkt/pull/13.patch
https://github.com/gavincarr/mod_auth_tkt/pull/13.diff


Reply to this email directly or view it on GitHub:
#13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants