BobinkCOpcUa/src/config.c, branch masterOPC UA applications in Chttp://git.tvcloud.fr/BobinkCOpcUa/atom/src/config.c?h=master2026-02-18T23:01:18ZRefactor: reduce duplication and tighten helpers2026-02-18T23:01:18ZThomas Vanbesientvanbesi@proton.me2026-02-18T23:01:18Zurn:sha1:f3648fefe040152bb1676d651ebf7d836cb8ac9e
- Remove redundant applicationUri log in print_application_description
- Use UA_SECURITY_POLICY_NONE_URI macro instead of hardcoded string
- Extract _s_register_with_lds / _s_deregister_from_lds helpers
- Rename signal handler param 'sign' to 'sig' for consistency
- Add INT_MIN/INT_MAX bounds check to config_require_int
- Extract shared test helpers into tests/test_helpers.sh
Rename all identifiers to strict Linux snake_case2026-02-18T22:23:44ZThomas Vanbesientvanbesi@proton.me2026-02-18T22:23:44Zurn:sha1:3d30c8499ae37ca0ff837e9deaad359de0297765
Types PascalCase→snake_case, functions camelCase→snake_case,
static functions get _s_ prefix, globals get g_ prefix,
struct members and locals to snake_case.
Fix memory leak, add volatile, reduce duplication2026-02-18T09:00:46ZThomas Vanbesientvanbesi@proton.me2026-02-18T09:00:46Zurn:sha1:9fe3814a2ef0be8e5b693fb0fa42064b33d0ae45
- config.c: free partial strdup on configAppend failure
- common.c: consolidate loadTrustStore error paths with goto
- server_lds.c, server_register.c: make running volatile, remove
non-async-signal-safe call from signal handler
- server_register.c: extract LdsClientParams + makeLdsClientConfig
to deduplicate the register/deregister client config setup
Replace ClientFindServers with unified Client, use trust store directories2026-02-17T18:06:22ZThomas Vanbesientvanbesi@proton.me2026-02-17T18:06:22Zurn:sha1:827e90e0daabe32e058e08dd2a253425898a7e7a
Replace the single-purpose ClientFindServers program with a unified Client
that supports three operations via CLI: find-servers, get-endpoints, and
read-time. This simplifies the architecture by using one client binary with
a single config file instead of a monolithic program that did everything in
one run.
Split the ServerRegister config into separate server and client config files
so the LDS-registration credentials are isolated from the server's own
settings. The discovery URL moves from config to a CLI argument.
Replace repeated trustList config entries with a single trustStore directory
path. Each program now points to a directory under certs/trust/ containing
.der files, so adding or removing trust is a file-copy operation rather than
editing every config file. Add loadTrustStore()/freeTrustStore() to
common.c and remove the now-unused configGetAll() from the config parser.
Simplify the test matrix from 6 to 4 cases (security and auth are
orthogonal, so the full 3x2 matrix is unnecessary). Update run_test.sh to
invoke the new Client three times and use port-polling instead of sleep.
Replace CLI arguments with config-file parser and add integration tests2026-02-17T10:07:37ZThomas Vanbesientvanbesi@proton.me2026-02-17T10:07:37Zurn:sha1:a54421dd976fd8081e96c11c2621076876c9986b
Introduce a reusable key=value config parser (config.h/c) and convert
all three programs to read their settings from config files instead of
positional command-line arguments. Add example config files in config/
and 6 CTest integration tests covering None/Basic256Sha256/Aes128 with
anonymous and user authentication. Remove the now-obsolete launch.sh.