)]}'
{"id":"openvpn~528","triplet_id":"openvpn~master~Ie147f81e5990b8292be090fd05c23b91f8e308d4","project":"openvpn","branch":"master","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":"2024-02-19 12:39:43.000000000","reason":"Change was abandoned"},"1000034":{"account":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"last_update":"2024-02-19 12:39:43.000000000","reason":"Change was abandoned"},"1000001":{"account":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},"last_update":"2024-02-16 16:06:56.000000000","reason":"\u003cGERRIT_ACCOUNT_1000001\u003e replied on the change","reason_account":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."}}},"hashtags":[],"change_id":"Ie147f81e5990b8292be090fd05c23b91f8e308d4","subject":"Route: remove incorrect routes on exit","status":"ABANDONED","created":"2024-02-16 15:50:35.000000000","updated":"2024-02-19 12:39:43.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a7d9c6c2c9d770a6d774a4a86ca2eb3e7f0b163d","_number":528,"virtual_id_number":528,"owner":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"actions":{},"labels":{"Code-Review":{"rejected":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},{"value":-2,"date":"2024-02-16 16:06:56.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."}],"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,"blocking":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},{"_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":"2024-02-16 15:50:35.000000000","updated_by":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"reviewer":{"_account_id":1000026,"name":"openvpn-devel","email":"openvpn-devel@lists.sourceforge.net","username":"openvpn-devel"},"state":"CC"},{"updated":"2024-02-16 15:50:35.000000000","updated_by":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"reviewer":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},"state":"REVIEWER"},{"updated":"2024-02-16 15:50:35.000000000","updated_by":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"reviewer":{"_account_id":1000003,"name":"plaisthos","display_name":"Arne Schwabe","email":"arne-openvpn@rfc2549.org","username":"plaisthos"},"state":"REVIEWER"}],"messages":[{"id":"fd75651a12eabf1dd6ee84af6c31f5bfdf50115a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"date":"2024-02-16 15:50:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b66ff505dc5e124d36fb9426201382e7ad6cf760","author":{"_account_id":1000001,"name":"flichtenheld","display_name":"Frank Lichtenheld","email":"frank@lichtenheld.com","username":"flichtenheld","status":"OpenVPN Inc."},"date":"2024-02-16 16:06:56.000000000","message":"Patch Set 1: Code-Review-2\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"a7d9c6c2c9d770a6d774a4a86ca2eb3e7f0b163d","tag":"autogenerated:gerrit:abandon","author":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"date":"2024-02-19 12:39:43.000000000","message":"Abandoned\n\nDuplicate with wrong change-id","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"75d246fdad940d6c7d9d9ed49f648acb216c2dd8","revisions":{"75d246fdad940d6c7d9d9ed49f648acb216c2dd8":{"kind":"REWORK","_number":1,"created":"2024-02-16 15:50:35.000000000","uploader":{"_account_id":1000034,"name":"its_Giaan","display_name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","username":"its_Giaan"},"ref":"refs/changes/28/528/1","fetch":{"anonymous http":{"url":"http://gerrit.openvpn.net/openvpn","ref":"refs/changes/28/528/1","commands":{"Branch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/28/528/1 \u0026\u0026 git checkout -b change-528 FETCH_HEAD","Checkout":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/28/528/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/28/528/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/28/528/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull http://gerrit.openvpn.net/openvpn refs/changes/28/528/1","Reset To":"git fetch http://gerrit.openvpn.net/openvpn refs/changes/28/528/1 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"7869617a0f85089fb5e6fbe2db6f03542a8f33f4","subject":"Ensure that all unit tests use unbuffered stdout and stderr"}],"author":{"name":"Gianmarco De Gregori","email":"gianmarco@mandelbit.com","date":"2023-04-07 11:05:15.000000000","tz":120},"committer":{"name":"itsGiaan","email":"gianmarcod95@outlook.com","date":"2024-02-16 15:50:28.000000000","tz":60},"subject":"Route: remove incorrect routes on exit","message":"Route: remove incorrect routes on exit\n\nImplemented a safeguard to verify the returned value\nfrom add_route3() when the default gateway is not a local\nremote host.\n\nPrior to this implementation, RT_DID_LOCAL flag was\nerroneously set even in case of add_route3() failure.\nThis problem typically occurs when there\u0027s no default\nroute and the --redirect-gateway def1 option is specified,\nand in case of reconnection makes it impossible for the client\nto reobtain the route to the server.\nThis fix ensures OpenVPN accurately deletes the appropriate\nroute on exit by properly handling add_route3() return value.\n\nFixes: Trac #1457\nChange-Id: Ie147f81e5990b8292be090fd05c23b91f8e308d4\nSigned-off-by: Gianmarco De Gregori \u003cgianmarco@mandelbit.com\u003e\n"},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}