)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"change_message_id":"b237105e1682851efb7845e7dde7126a3be472eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2dd43878_14973cd8","updated":"2025-10-28 13:39:17.000000000","message":"Looks good, but this API (PathCchCanonicalizeEx) is available starting from Windows 8. Let\u0027s merge it for master only - by default config_dir comes from the installer and terminates with \"\\\", and you need admin rights to change it. So I think it is not that critical.","commit_id":"71aa973dea66d29f1020151fbd6bdd9598cb195b"}],"src/openvpnserv/validate.c":[{"author":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"change_message_id":"22c3d94483c39a2c1179ea183f77b918382133eb","unresolved":true,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"#ifndef HAVE_PATHCCH_ENSURE_TRAILING_SLASH"},{"line_number":31,"context_line":"#define PATHCCH_ENSURE_TRAILING_SLASH 0x20"},{"line_number":32,"context_line":"#endif"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"static const WCHAR *white_list[] \u003d {"},{"line_number":35,"context_line":"    L\"auth-retry\","}],"source_content_type":"text/x-csrc","patch_set":1,"id":"4804eff4_88a77403","line":32,"updated":"2025-10-27 21:13:04.000000000","message":"Any reason that we cannot just do #ifndef PATHCCH_ENSURE_TRAILING_SLASH here and have to do the cmake dance?","commit_id":"71aa973dea66d29f1020151fbd6bdd9598cb195b"},{"author":{"_account_id":1000009,"name":"selvanair","display_name":"Selva Nair","email":"selva.nair@gmail.com","username":"selvanair"},"change_message_id":"9eea84327c00bb56740fdb851f750ed9fdaca0cf","unresolved":true,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"#ifndef HAVE_PATHCCH_ENSURE_TRAILING_SLASH"},{"line_number":31,"context_line":"#define PATHCCH_ENSURE_TRAILING_SLASH 0x20"},{"line_number":32,"context_line":"#endif"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"static const WCHAR *white_list[] \u003d {"},{"line_number":35,"context_line":"    L\"auth-retry\","}],"source_content_type":"text/x-csrc","patch_set":1,"id":"d1f80050_27c71e51","line":32,"in_reply_to":"4804eff4_88a77403","updated":"2025-10-27 21:56:55.000000000","message":"When its defined, its an enum in pathcch.h, not a define. mingw does have it now, but apparently not in ubuntu 24.04 that we use.","commit_id":"71aa973dea66d29f1020151fbd6bdd9598cb195b"},{"author":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"change_message_id":"43ad65b6fbf3fa80143f91b20d64168fa5b47c61","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"#ifndef HAVE_PATHCCH_ENSURE_TRAILING_SLASH"},{"line_number":31,"context_line":"#define PATHCCH_ENSURE_TRAILING_SLASH 0x20"},{"line_number":32,"context_line":"#endif"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"static const WCHAR *white_list[] \u003d {"},{"line_number":35,"context_line":"    L\"auth-retry\","}],"source_content_type":"text/x-csrc","patch_set":1,"id":"a259ecd3_08086e9f","line":32,"in_reply_to":"d1f80050_27c71e51","updated":"2025-10-27 22:11:14.000000000","message":"Acknowledged","commit_id":"71aa973dea66d29f1020151fbd6bdd9598cb195b"}]}
