Re: [PATCH 2/2] Redirect Apache status requests

[Matthias Fischer <matthias.fischer@ipfire.org>]

On 22.06.2026 22:42, Peer Dietzmann wrote:
> Hi Matthias,

Hi Peer,
> thank you for adding this. Will try it again when it was merged but it 
> looks like the tests I did today, so it should work.

Sorry, but this isn't clear to me - blame the weather... ;-)

This patch hasn't been pushed, he is NOT in patchwork or GIT.
Shall I push it or will you?

Or does your message mean that it is no longer needed?

Best
Matthias

> Best,
> 
> Peer
> 
> On 22/06/2026 19:18, Matthias Fischer wrote:
>> *Third try - fixed typo in patch line*
>>
>> - "-Np1" should have been "-Np2".
>>
>> Without the quotes, of course.
>>
>> Sorry...its too warm and the observium-lfs has a different path variable
>> (DIR_APP) than others. I put both files in attachments.
>>
>> ##########################################################
>>
>> *Second try with attachment in unix format (line breaks)!*
>>
>> ##########################################################
>>
>> On 22.06.2026 16:20, Peer Dietzmann wrote:
>>> Hello,
>> Hi,
>>
>>> there is also another patch required to make the module working. But I
>>> just can't find the file in the repository that needs to be changed:
>> You won't find this file in the repository - that would be too easy. ;-)
>>
>> This 'apache'-file is stored in the observium archive
>> (observium/scripts/agent-local/apache). And only there. It is extracted
>> during the build phase, and is then immediately deleted. This is
>> controlled via the lfs file:
>>
>> Line 83 (to be sure: first delete and then unpack source):
>> ...
>> @rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar axf $(DIR_DL)/$(DL_FILE)
>> ...
>>
>> Line 103 (clean up):
>> ...
>> @rm -rf $(DIR_APP)
>> ...
>>
>> IMHO the best way to apply your changes would be a patchfile (see
>> attachment) which needs to be copied into the repository in the
>> directory '/src/patches/observium', e.g.).
>>
>> Then the lfs file needs something like this in a new line *84*:
>> ...
>> cd $(DIR_APP) && patch -Np2 <
>> $(DIR_SRC)/src/patches/observium/[NAME_OF_PATCHFILE]
>> ...
>> HTH
>> Matthias
>>
>>> Line 29 of /usr/lib/observium_agent/scripts-available/apache has to be
>>> changed to:
>>>
>>>           close INFILE;
>>> } else {
>>>           # grab the status URL (fresh data)
>>>           @data = split /(\n)/, LWP::Simple::get(
>>> 'http://localhost:81/server-status?auto' )
>>>                   or die "Data fetch failure.\n";
>>>
>>>           # write file
>>>
>>> The *localhost* has to be changed to *localhost:81*
>>>
>>> The concerned bug that should be fixed through those three patches is:
>>> https://bugzilla.ipfire.org/show_bug.cgi?id=14017
>>>
>>> Best,
>>>
>>> Peer
>>>
>>> On 22/06/2026 16:15, Peer Dietzmann wrote:
>>>> Redirect requests to Apache's status to it's backend, force local requests.
>>>> This is required to allow Observium's agent to pull metrics from Apache.
>>>>
>>>> Best,
>>>> Peer
>>>>
>>>> Signed-off-by: Peer Dietzmann <dietzmann@brecht-schule.hamburg>
>>>> ---
>>>>    config/httpd/httpd.conf | 6 ++++++
>>>>    1 file changed, 6 insertions(+)
>>>>
>>>> diff --git a/config/httpd/httpd.conf b/config/httpd/httpd.conf
>>>> index c694bffe2..09b10b50f 100644
>>>> --- a/config/httpd/httpd.conf
>>>> +++ b/config/httpd/httpd.conf
>>>> @@ -49,4 +49,10 @@ Include /etc/httpd/conf/default-server.conf
>>>>    # Virtual server configuration
>>>>    Include /etc/httpd/conf/vhosts.d/*.conf
>>>>    
>>>> +# Allow local clients to pull Apache's status
>>>> +<Location /server-status>
>>>> +    SetHandler server-status
>>>> +    Require local
>>>> +</Location>
>>>> +
>>>>    # EOF
>