[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#55055] Fwd: [bug#55055] [PATCH] gnu: wireguard: Add support for Pre
From: |
Maxime Devos |
Subject: |
[bug#55055] Fwd: [bug#55055] [PATCH] gnu: wireguard: Add support for PresharedKey |
Date: |
Thu, 21 Apr 2022 23:55:17 +0200 |
User-agent: |
Evolution 3.38.3-1 |
Paul Alesius schreef op do 21-04-2022 om 22:41 [+0200]:
> > * WDYT of verifying that the preshared key looks ‘reasonable’ (I
> > guess only a-z0-9 characters, no spaces or newlines, not a
> > bytevector ...)
>
> I could develop a subsystem for validating the fields of the
> wireguard but isn't it better to provide validations from the guix
> framework more broadly? With my level of Guile scripting right now, I
> doubt that it would be accepted.
There's already a basic system for this: field sanitisers. Have a look
at <network-address> and its 'assert-valid-address'. Long term, there
were some ideas for a contract system à la racket, there was some e-
mail thread about that.
Also, some very basic validation could be replacing
(format #f "PresharedKey = ~a\n" preshared-key)
by
(string-append "PresharedKey = " preshared-key "\n")
-- basically, let string-append do some basic type checking. This only
checks that it's a string though. 'make-regexp' and friends may be
useful for more complete validation.
Greetings,
Maxime.
signature.asc
Description: This is a digitally signed message part