fix(duniter-primitives): stricter idty name validation

Cargo.lock

@@ -827,6 +827,7 @@ dependencies = [
 name = "common-runtime"
 version = "0.8.0-dev"
 dependencies = [
+ "duniter-primitives",
  "frame-support",
  "frame-system",
  "log",
@@ -1347,6 +1348,17 @@ dependencies = [
  "tokio",
 ]
 
+[[package]]
+name = "duniter-primitives"
+version = "3.0.0"
+dependencies = [
+ "frame-support",
+ "parity-scale-codec",
+ "scale-info",
+ "sp-runtime",
+ "sp-std",
+]
+
 [[package]]
 name = "dyn-clonable"
 version = "0.9.0"

primitives/duniter/Cargo.toml

+[package]
+authors = ['librelois <c@elo.tf>']
+description = 'primitives for duniter runtime.'
+edition = '2018'
+homepage = 'https://substrate.dev'
+license = 'AGPL-3.0'
+name = 'duniter-primitives'
+readme = 'README.md'
+repository = 'https://git.duniter.org/nodes/rust/duniter-v2s'
+version = '3.0.0'
+
+[features]
+default = ['std']
+std = [
+    'codec/std',
+    'frame-support/std',
+	'sp-runtime/std',
+	'sp-std/std',
+]
+try-runtime = ['frame-support/try-runtime']
+
+[dependencies]
+
+# substrate
+scale-info = { version = "1.0", default-features = false, features = ["derive"] }
+
+[dependencies.codec]
+default-features = false
+features = ['derive']
+package = 'parity-scale-codec'
+version = '2.3.1'
+
+[dependencies.frame-support]
+default-features = false
+git = 'https://github.com/librelois/substrate.git'
+branch = 'duniter-monthly-2022-02'
+
+[dependencies.sp-runtime]
+default-features = false
+git = 'https://github.com/librelois/substrate.git'
+branch = 'duniter-monthly-2022-02'
+
+[dependencies.sp-std]
+default-features = false
+git = 'https://github.com/librelois/substrate.git'
+branch = 'duniter-monthly-2022-02'
+
+### DOC ###
+
+[package.metadata.docs.rs]
+targets = ['x86_64-unknown-linux-gnu']

primitives/duniter/src/lib.rs

+// Copyright 2021 Axiom-Team
+//
+// This file is part of Substrate-Libre-Currency.
+//
+// Substrate-Libre-Currency is free software: you can redistribute it and/or modify
+// it under the terms of the GNU Affero General Public License as published by
+// the Free Software Foundation, version 3 of the License.
+//
+// Substrate-Libre-Currency is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU Affero General Public License for more details.
+//
+// You should have received a copy of the GNU Affero General Public License
+// along with Substrate-Libre-Currency. If not, see <https://www.gnu.org/licenses/>.
+
+#![cfg_attr(not(feature = "std"), no_std)]
+
+/// Bound length; forbid trailing or double spaces; accept only ascii alphanumeric or punctuation or space
+pub fn validate_idty_name(idty_name: &[u8]) -> bool {
+    idty_name.len() >= 3
+        && idty_name.len() <= 64
+        && idty_name[0] != 32
+        && idty_name[idty_name.len() - 1] != 32
+        && idty_name
+            .iter()
+            .all(|c| c.is_ascii_alphanumeric() || c.is_ascii_punctuation() || *c == 32)
+        && idty_name
+            .iter()
+            .zip(idty_name.iter().skip(1))
+            .all(|(c1, c2)| *c1 != 32 || *c2 != 32)
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_validate_idty_name() {
+        assert!(validate_idty_name(b"B0b"));
+        assert!(validate_idty_name(b"lorem ipsum dolor-sit_amet."));
+        assert!(!validate_idty_name(b" space"));
+        assert!(!validate_idty_name(b"space "));
+        assert!(!validate_idty_name(b"double  space"));
+        assert!(!validate_idty_name("non-ascii🌵".as_bytes()));
+        assert!(!validate_idty_name("ğune".as_bytes()));
+    }
+}

runtime/common/Cargo.toml

@@ -20,6 +20,7 @@ runtime-benchmarks = [
 ]
 std = [
     'codec/std',
+    'duniter-primitives/std',
     'frame-support/std',
     'frame-system/std',
 	'log/std',
@@ -43,6 +44,7 @@ std = [
 ]
 
 [dependencies]
+duniter-primitives = { path = '../../primitives/duniter', default-features = false }
 pallet-authority-members = { path = '../../pallets/authority-members', default-features = false }
 pallet-certification = { path = '../../pallets/certification', default-features = false }
 pallet-duniter-account = { path = '../../pallets/duniter-account', default-features = false }

runtime/common/src/lib.rs

@@ -84,6 +84,6 @@ impl<Runtime: pallet_babe::Config> frame_support::pallet_prelude::Get<u64>
 pub struct IdtyNameValidatorImpl;
 impl pallet_identity::traits::IdtyNameValidator for IdtyNameValidatorImpl {
     fn validate(idty_name: &pallet_identity::IdtyName) -> bool {
-        idty_name.0.len() >= 3 && idty_name.0.len() <= 64
+        duniter_primitives::validate_idty_name(&idty_name.0)
     }
 }