From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.haj.ipfire.org (localhost [IPv6:::1]) by mail02.haj.ipfire.org (Postfix) with ESMTP id 4gl2x93Ftsz2xfM for ; Tue, 23 Jun 2026 11:32:45 +0000 (UTC) Received: from mail01.ipfire.org (mail01.haj.ipfire.org [172.28.1.202]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature ECDSA (secp384r1 raw public key) server-digest SHA384 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mail01.haj.ipfire.org", Issuer "YR2" (not verified)) by mail02.haj.ipfire.org (Postfix) with ESMTPS id 4gl2x61B9zz2xJ5 for ; Tue, 23 Jun 2026 11:32:42 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail01.ipfire.org (Postfix) with ESMTPSA id 4gl2x36HZ7z1Fh for ; Tue, 23 Jun 2026 11:32:39 +0000 (UTC) DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=ipfire.org; s=202003ed25519; t=1782214360; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/Ie6XTpNzbVziTbrdA2mOuh6xysO+IAJiCSMbE+JUWE=; b=Z9uCTeQ112efRYqe0nFUtWoQNet6S4gTfstuczlk1QEbRWRVnoj1WWi8ZfgyzC0fLipxFb K56Np9P0Z1punpBg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ipfire.org; s=202003rsa; t=1782214360; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/Ie6XTpNzbVziTbrdA2mOuh6xysO+IAJiCSMbE+JUWE=; b=R/Zdd5DVQn/quEE6UPoaC+HoFpHpb+JIPqScFjxL/5gk/raOpf0PSMmO+z8p8lv0siNDHj 5e6huuDwE+v8bIHoJeUCbKvAqlpKggb7qwkKS6/NFki7PFvRCH6r41Ku+hurl++aSXz3Za r8Ns2r8yymcgVPhi9LYAMdnT+L6Ktu0qv0XGkOCvf7dN/DFNy0XPK65cN8/wwgI/Q1qflu e5vvzzPwWt822lqa61jtaQTeWBmtfuGOCIyayJojVbbNyL9EJ9gz541ZEfXcJbZZbTUQCz eNkmvJeLrZHOz25C0Ey0VGe9zNQrBRsOijgmtlxjJPwzLFBU1Ncl08j6gVOORw== Message-ID: <32e4bd79-effd-4720-bb44-aa8eefda9a83@ipfire.org> Date: Tue, 23 Jun 2026 13:32:33 +0200 Precedence: list List-Id: List-Subscribe: , List-Unsubscribe: , List-Post: List-Help: Sender: Mail-Followup-To: MIME-Version: 1.0 Subject: Re: [PATCH 2/2] Redirect Apache status requests To: development@lists.ipfire.org References: <20260622141502.375427-1-dietzmann@brecht-schule.hamburg> <03c95271-dec0-4b62-868c-40c7fb25bbb8@brecht-schule.hamburg> <49784f88-adb5-410f-8b69-f2bf2007a3d9@ipfire.org> <9f83c1ba-02e0-42e4-a350-c4264d53f123@ipfire.org> <10436d11-146d-4358-94b5-2687f577bed9@brecht-schule.hamburg> Content-Language: en-US From: Matthias Fischer In-Reply-To: <10436d11-146d-4358-94b5-2687f577bed9@brecht-schule.hamburg> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 23.06.2026 11:21, Peer Dietzmann wrote: > Hi Matthias, Hi Peer, > On 23/06/2026 09:39, Matthias Fischer wrote: >> 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? > > now I'm a bit confused, both patches I sent, must be merged into the > next release to make it work. The patch you provided also. If you know > where this is done, please do it, all three are required. What I meant > is, that when everything is in testing, then I will test it again to > confirm that the changed files are shipped correctly. Ok, sorry for the noise, but in fact it was me who was a bit confused. As I wrote, its the weather to blame... ;-) I just wasn't sure if you'd like to push the third patch yourself. I'll prepare a patch and will send it in a few minutes, no problem. Best Matthias > Best, > > Peer > >> >> 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 >>>>>> --- >>>>>> 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 >>>>>> + >>>>>> + SetHandler server-status >>>>>> + Require local >>>>>> + >>>>>> + >>>>>> # EOF >>