From: "Adolf Belka (ipfire-dev)" <adolf.belka@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: Feedback on Core Update 154 testing
Date: Wed, 17 Feb 2021 14:21:03 +0100 [thread overview]
Message-ID: <e0fff0dd-9be2-4813-d93e-ad97754390cf@ipfire.org> (raw)
In-Reply-To: <92699267-80d9-111e-ef6b-06c2a04af9fc@ipfire.org>
[-- Attachment #1: Type: text/plain, Size: 1292 bytes --]
Hi All,
I confirm that the problem of the curly bracket raised below is now fixed in the latest version of Core Update 154.
I have also tested Core Update 154 out in my VM Testbed and have gone through every menu item and checked everything out that I can. The things I could not test were IPsec Tunnel, URL Filter, Update Accelerator, Captive Portal.
All the rest I have been able to turn something on and set things and see if there were any problems. It is not a really deep testing of the menu items but I didn't find any issues at that level.
Regards,
Adolf.
On 16/02/2021 13:59, Adolf Belka (ipfire-dev) wrote:
> Hi All,
>
> After working on a new bug (#10628), my code was not working as I expected it to. After investigating I realised that when I provided https://patchwork.ipfire.org/patch/3724/ to add Deny Known Client capability to dhcp.cgi I had placed a curly bracket in the wrong place.
>
> It is not syntactically wrong so there is no Internal Error message coming up but if the Deny Known Clients is not checked then there is no check made for a valid Default Lease Time.
>
>
> I made an error when I inserted my code into dhcp.cgi for the Deny Known Clients fix.
>
>
> I will raise a patch to correct this.
>
>
> Regards,
>
> Adolf.
>
prev parent reply other threads:[~2021-02-17 13:21 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-16 12:59 Adolf Belka (ipfire-dev)
2021-02-16 15:02 ` Michael Tremer
2021-02-17 13:21 ` Adolf Belka (ipfire-dev) [this message]
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=e0fff0dd-9be2-4813-d93e-ad97754390cf@ipfire.org \
--to=adolf.belka@ipfire.org \
--cc=development@lists.ipfire.org \
/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