)]}'
{"id":"openvpn~1185","triplet_id":"openvpn~release%2F2.6~I9b9f38e11b06701142bdc1339d9bedf080de5f86","project":"openvpn","branch":"release/2.6","attention_set":{},"removed_from_attention_set":{"1000003":{"account":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"last_update":"2025-09-17 16:47:20.000000000","reason":"Change was submitted"},"1000002":{"account":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"last_update":"2025-09-17 09:06:31.000000000","reason":"removed on reply"},"1000001":{"account":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},"last_update":"2025-09-17 16:47:20.000000000","reason":"Change was submitted"},"1000008":{"account":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"last_update":"2025-09-17 16:47:20.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I9b9f38e11b06701142bdc1339d9bedf080de5f86","subject":"openvpnserv: Fix writing messages to the event log","status":"MERGED","created":"2025-09-15 11:07:02.000000000","updated":"2025-09-17 16:47:20.000000000","submitted":"2025-09-17 16:47:20.000000000","submitter":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"total_comment_count":1,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"1185","meta_rev_id":"210b2cd313bbda9b5eb0cb36c9c6a933c4843812","_number":1185,"virtual_id_number":1185,"owner":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"actions":{},"labels":{"Code-Review":{"all":[{"value":0,"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"}],"values":{"-2":"This shall not be submitted","-1":"I would prefer this is not submitted as is"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me, approved"},"default_value":0}},"removable_reviewers":[{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."}],"reviewers":{"REVIEWER":[{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"}],"CC":[{"_account_id":1000026,"name":"openvpn-devel","email":"openvpn-devel@lists.sourceforge.net","username":"openvpn-devel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-09-15 11:07:03.000000000","updated_by":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"reviewer":{"_account_id":1000026,"name":"openvpn-devel","email":"openvpn-devel@lists.sourceforge.net","username":"openvpn-devel"},"state":"CC"},{"updated":"2025-09-15 11:07:03.000000000","updated_by":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"reviewer":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},"state":"REVIEWER"},{"updated":"2025-09-15 11:07:03.000000000","updated_by":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"reviewer":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"state":"REVIEWER"},{"updated":"2025-09-15 12:48:41.000000000","updated_by":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"reviewer":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"state":"REVIEWER"}],"messages":[{"id":"a2aeb74e480fa2a615c678fc92cde12c67f4da1a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"date":"2025-09-15 11:07:02.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"1919443ab158d24cfcd13ef818b246bbced8ce4a","author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"date":"2025-09-15 12:48:41.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"e306e84c9df112b95bde0bdeca8f3cfeeac3a2e8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"date":"2025-09-15 13:30:55.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Code-Review+1 (copy condition: \"changekind:NO_CHANGE OR changekind:TRIVIAL_REBASE OR is:MIN\")\n","accounts_in_message":[],"_revision_number":2},{"id":"2efa37c3fc121647fb19ec10a68439b3e016bca7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"date":"2025-09-16 07:05:41.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"4c34a22752ecacf443345e4f3fff8493960e0c96","author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"date":"2025-09-17 09:06:31.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"210b2cd313bbda9b5eb0cb36c9c6a933c4843812","tag":"autogenerated:gerrit:merged","author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"date":"2025-09-17 16:47:20.000000000","message":"Change has been successfully pushed.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"cabbf49ba267477c3015c3da9ee4bd45b1207211","revisions":{"d7a12468854a61627dfa9bc601c944d1f0f813f9":{"kind":"REWORK","_number":1,"created":"2025-09-15 11:07:02.000000000","uploader":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"ref":"refs/changes/85/1185/1","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/85/1185/1","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/1 \u0026\u0026 git checkout -b change-1185 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/85/1185/1","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/1 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"d383d6ed6270b8d1f95716d08e9da3dd0d712f2d","subject":"win: replace wmic invocation with powershell"}],"author":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-15 09:05:14.000000000","tz":180},"committer":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-15 11:05:05.000000000","tz":180},"subject":"openvpnserv: Fix writing messages to the event log","message":"openvpnserv: Fix writing messages to the event log\n\nThere are two problems with the current implementation:\n\n - due to the code bug, we never display actual error message\ncorresponding to the Windows error code. We use\nFORMAT_MESSAGE_ALLOCATE_BUFFER, in which case we must pass\na pointer to the LPTSTR, not the LPTSTR itself.\n\n - The error is not displayed in the \"General\" tab, which is very confusing.\nOne needs to go to the \"Details\" tab to see what is wrong.\n\nThis commit solves both problems. We now display a proper error\nmessage in addition to the text provided by the service (\"what went wrong\").\nWhile on it, remove trailing symbols ín a safer way.\n\nTo display the message in \"General\" tab, we create a registered message file\n(openvpnservmsg.dll), which contains message template. Note that this requires\nchanges to the installer - we need to install the new DLL and\nadd a registry entry.\n\nGitHub: https://github.com/OpenVPN/openvpn/issues/842\n\nChange-Id: I9b9f38e11b06701142bdc1339d9bedf080de5f86\nSigned-off-by: Lev Stipakov \u003clev@openvpn.net\u003e\n"},"branch":"refs/heads/release/2.6"},"db8eb209b48731fb9ef32601a0e3c5beefc09be8":{"kind":"REWORK","_number":2,"created":"2025-09-15 13:30:55.000000000","uploader":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"ref":"refs/changes/85/1185/2","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/85/1185/2","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/2 \u0026\u0026 git checkout -b change-1185 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/85/1185/2","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/2 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"d383d6ed6270b8d1f95716d08e9da3dd0d712f2d","subject":"win: replace wmic invocation with powershell"}],"author":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-15 09:05:14.000000000","tz":180},"committer":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-15 13:30:43.000000000","tz":180},"subject":"openvpnserv: Fix writing messages to the event log","message":"openvpnserv: Fix writing messages to the event log\n\nThere are two problems with the current implementation:\n\n - due to the code bug, we never display actual error message\ncorresponding to the Windows error code. We use\nFORMAT_MESSAGE_ALLOCATE_BUFFER, in which case we must pass\na pointer to the LPTSTR, not the LPTSTR itself.\n\n - The error is not displayed in the \"General\" tab, which is very confusing.\nOne needs to go to the \"Details\" tab to see what is wrong.\n\nThis commit solves both problems. We now display a proper error\nmessage in addition to the text provided by the service (\"what went wrong\").\nWhile on it, remove trailing symbols ín a safer way.\n\nTo display the message in \"General\" tab, we create a registered message file\n(openvpnservmsg.dll), which contains message template. Note that this requires\nchanges to the installer - we need to install the new DLL and\nadd a registry entry.\n\nGitHub: https://github.com/OpenVPN/openvpn/issues/842\n\nChange-Id: I9b9f38e11b06701142bdc1339d9bedf080de5f86\nSigned-off-by: Lev Stipakov \u003clev@openvpn.net\u003e\n"},"branch":"refs/heads/release/2.6"},"05cecb719ad2ec4dd839a860595154b347ea339e":{"kind":"REWORK","_number":3,"created":"2025-09-16 07:05:41.000000000","uploader":{"_account_id":1000008,"name":"stipa","display_name":"Lev Stipakov","email":"lstipakov@gmail.com","username":"stipa"},"ref":"refs/changes/85/1185/3","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/85/1185/3","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/3 \u0026\u0026 git checkout -b change-1185 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/85/1185/3","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/3 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"d383d6ed6270b8d1f95716d08e9da3dd0d712f2d","subject":"win: replace wmic invocation with powershell"}],"author":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-15 09:05:14.000000000","tz":180},"committer":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-16 07:04:35.000000000","tz":180},"subject":"openvpnserv: Fix writing messages to the event log","message":"openvpnserv: Fix writing messages to the event log\n\nThere are two problems with the current implementation:\n\n - due to the code bug, we never display actual error message\ncorresponding to the Windows error code. We use\nFORMAT_MESSAGE_ALLOCATE_BUFFER, in which case we must pass\na pointer to the LPTSTR, not the LPTSTR itself.\n\n - The error is not displayed in the \"General\" tab, which is very confusing.\nOne needs to go to the \"Details\" tab to see what is wrong.\n\nThis commit solves both problems. We now display a proper error\nmessage in addition to the text provided by the service (\"what went wrong\").\nWhile on it, remove trailing symbols ín a safer way.\n\nTo display the message in \"General\" tab, we create a registered message file\n(openvpnservmsg.dll), which contains message template. Note that this requires\nchanges to the installer - we need to install the new DLL and\nadd a registry entry.\n\nGitHub: https://github.com/OpenVPN/openvpn/issues/842\n\nChange-Id: I9b9f38e11b06701142bdc1339d9bedf080de5f86\nSigned-off-by: Lev Stipakov \u003clev@openvpn.net\u003e\n"},"branch":"refs/heads/release/2.6"},"cabbf49ba267477c3015c3da9ee4bd45b1207211":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":4,"created":"2025-09-17 16:47:20.000000000","uploader":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"ref":"refs/changes/85/1185/4","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/85/1185/4","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/4 \u0026\u0026 git checkout -b change-1185 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/85/1185/4","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/85/1185/4 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"68c01720eecc1772b3f648b9e043e396d943f632","subject":"Check message id/acked ids too when doing sessionid cookie checks"}],"author":{"name":"Lev Stipakov","email":"lev@openvpn.net","date":"2025-09-17 09:06:44.000000000","tz":120},"committer":{"name":"Gert Doering","email":"gert@greenie.muc.de","date":"2025-09-17 16:37:17.000000000","tz":120},"subject":"openvpnserv: Fix writing messages to the event log","message":"openvpnserv: Fix writing messages to the event log\n\nThere are two problems with the current implementation:\n\n - due to the code bug, we never display actual error message\ncorresponding to the Windows error code. We use\nFORMAT_MESSAGE_ALLOCATE_BUFFER, in which case we must pass\na pointer to the LPTSTR, not the LPTSTR itself.\n\n - The error is not displayed in the \"General\" tab, which is very confusing.\nOne needs to go to the \"Details\" tab to see what is wrong.\n\nThis commit solves both problems. We now display a proper error\nmessage in addition to the text provided by the service (\"what went wrong\").\nWhile on it, remove trailing symbols ín a safer way.\n\nTo display the message in \"General\" tab, we create a registered message file\n(openvpnservmsg.dll), which contains message template. Note that this requires\nchanges to the installer - we need to install the new DLL and\nadd a registry entry.\n\nGitHub: https://github.com/OpenVPN/openvpn/issues/842\n\nChange-Id: I9b9f38e11b06701142bdc1339d9bedf080de5f86\nSigned-off-by: Lev Stipakov \u003clev@openvpn.net\u003e\nAcked-by: Gert Doering \u003cgert@greenie.muc.de\u003e\nGerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1185\nMessage-Id: \u003c20250917090653.25510-1-gert@greenie.muc.de\u003e\nURL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg33008.html\nSigned-off-by: Gert Doering \u003cgert@greenie.muc.de\u003e\n(backported from commit 06919a60ae61d6d88546b23b52092f742599a8ae)\n"},"branch":"refs/heads/release/2.6"}},"requirements":[],"submit_records":[],"submit_requirements":[]}