Bug 62497 - Make r.ap_auth_type writable
Summary: Make r.ap_auth_type writable
Status: NEW
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_lua (show other bugs)
Version: 2.4.33
Hardware: PC FreeBSD
: P2 enhancement (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-27 14:33 UTC by Michael Osipov
Modified: 2019-10-07 07:58 UTC (History)
1 user (show)



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Osipov 2018-06-27 14:33:27 UTC
I've configured a LuaHookLog to set r.user from a downstream host, but cannot set r.ap_auth_type to make it akin to a native Apache module authentication.

Here is my usecase: https://bz.apache.org/bugzilla/show_bug.cgi?id=62496

Looking at the C code, makes it pretty simple to implement.
Comment 1 Michael Osipov 2019-08-05 10:43:01 UTC
Shall I create a patch for that?
Comment 2 Christophe JAILLET 2019-08-17 19:50:16 UTC
Hi Mickael.

Patch proposals are always welcome :)
Comment 3 Michael Osipov 2019-08-18 10:32:14 UTC
(In reply to Christophe JAILLET from comment #2)
> Hi Mickael.
> 
> Patch proposals are always welcome :)

Great, I will try to work out a PR for this.
Comment 4 Michael Osipov 2019-08-19 12:16:54 UTC
PR created: https://github.com/apache/httpd/pull/63

This is how it looks in my access log now:
2019-08-19T14:06:03.070 147.54.64.149 michaelo@apache.org SPNEGO XVqQqzIujz1LpmpcW8M9NAAAAFg "POST /prod/api/documents HTTP/1.1" 201 - 863


After this is merged, I will create another issue to make this usable in mod_log_config via %at (auth_type) as a companion for %u.
Comment 5 Michael Osipov 2019-10-07 07:58:52 UTC
PR 63 is obsolete because trunk has been modified too much.

A new PR has been created: https://github.com/apache/httpd/pull/67