From 4f790cb64620b26f6236054bf56c9f67df3e670b Mon Sep 17 00:00:00 2001
From: hanishkvc <hanishkvc@gmail.com>
Date: Fri, 5 Dec 2025 20:47:33 +0530
Subject: [PATCH] SimpleSallap:SimpleProxy:DataClass Config - P3

Add a helper base class to try map data class's attributes into
underlying dict.

TODO: this potentially duplicates data in both normal attribute
space as well as dict items space. And will require additional
standard helper logics to be overridden to ensure sync between
both space etal. Rather given distance from python internals for
a long time now, on pausing and thinking a bit, better to move
into a simpler arch where attributes are directly worked on for
dict [] style access.
---
 .../public_simplechat/local.tools/config.py   | 36 +++++++++++++------
 .../local.tools/simpleproxy.json              |  6 ++--
 2 files changed, 28 insertions(+), 14 deletions(-)

diff --git a/tools/server/public_simplechat/local.tools/config.py b/tools/server/public_simplechat/local.tools/config.py
index 0fe797a3ac..682bde1229 100644
--- a/tools/server/public_simplechat/local.tools/config.py
+++ b/tools/server/public_simplechat/local.tools/config.py
@@ -2,7 +2,7 @@
 # by Humans for All
 #
 
-from dataclasses import dataclass, field
+from dataclasses import dataclass, field, fields
 from typing import Any, Optional
 import http.server
 import ssl
@@ -11,11 +11,25 @@ import urlvalidator as mUV
 import debug as mDebug
 
 
-gConfigNeeded = [ 'acl.schemes', 'acl.domains', 'sec.bearer' ]
+gConfigNeeded = [ 'acl.schemes', 'acl.domains', 'sec.bearerAuth' ]
+
+@dataclass
+class DictDataclassMixin(dict):
+    """
+    Mixin to ensure dataclass attributes get mapped as dict keys in __post_init__
+    """
+
+    def __post_init__(self):
+        """
+        Skip if already set using maybe say dict init mechanism or ...
+        """
+        for f in fields(self):
+            if f.name not in self:
+                self[f.name] = getattr(self, f.name)
 
 
-@dataclass(frozen=True)
-class Sec(dict):
+@dataclass
+class Sec(DictDataclassMixin, dict):
     """
     Used to store security related config entries
     """
@@ -24,17 +38,17 @@ class Sec(dict):
     bearerAuth: str = ""
 
 @dataclass
-class ACL(dict):
-    schemes: Optional[list[str]] = None
+class ACL(DictDataclassMixin, dict):
+    schemes: list[str] = field(default_factory=list)
     domains: list[str] = field(default_factory=list)
 
 @dataclass
-class Network(dict):
+class Network(DictDataclassMixin, dict):
     port: int = 3128
     addr: str = ''
 
 @dataclass
-class Op(dict):
+class Op(DictDataclassMixin, dict):
     configFile: str = "/dev/null"
     debug: bool = False
     server: http.server.ThreadingHTTPServer|None = None
@@ -43,7 +57,7 @@ class Op(dict):
     bearerTransformedYear: str = ""
 
 @dataclass
-class Config(dict):
+class Config(DictDataclassMixin, dict):
     op: Op = field(default_factory=Op)
     sec: Sec = field(default_factory=Sec)
     acl: ACL = field(default_factory=ACL)
@@ -104,7 +118,7 @@ class Config(dict):
                         exit(112)
                     self.set_value(cfg, gotValue)
                 except KeyError:
-                    print(f"ERRR:LoadConfig:{cfg}:UnknownCommand")
+                    print(f"ERRR:LoadConfig:{cfg}:UnknownConfig!")
                     exit(113)
 
     def process_args(self, args: list[str]):
@@ -142,7 +156,7 @@ class Config(dict):
                 if cArg == 'op.configFile':
                     self.load_config(aValue)
             except KeyError:
-                print(f"ERRR:ProcessArgs:{iArg}:{cArg}:UnknownCommand:{sys.exception()}")
+                print(f"ERRR:ProcessArgs:{iArg}:{cArg}:UnknownArgCommand!:{sys.exception()}")
                 exit(103)
         print(self)
         self.validate()
diff --git a/tools/server/public_simplechat/local.tools/simpleproxy.json b/tools/server/public_simplechat/local.tools/simpleproxy.json
index 77261bd1a2..1e3dcc67fc 100644
--- a/tools/server/public_simplechat/local.tools/simpleproxy.json
+++ b/tools/server/public_simplechat/local.tools/simpleproxy.json
@@ -1,10 +1,10 @@
 {
-    "allowed.schemes": [
+    "acl.schemes": [
         "file",
         "http",
         "https"
     ],
-    "allowed.domains": [
+    "acl.domains": [
         ".*\\.wikipedia\\.org$",
         ".*\\.bing\\.com$",
         "^www\\.bing\\.com$",
@@ -51,5 +51,5 @@
         "^github\\.com$",
         ".*\\.github\\.com$"
     ],
-    "bearer.insecure": "NeverSecure"
+    "sec.bearerAuth": "NeverSecure"
 }