[S] Change in openvpn[master]: SIGHUP-restart instance on UDP write error EADDRNOTAVAIL
"plaisthos \(Code Review\) via Openvpn-devel" <[email protected]> Mon, 3 Aug 2026 10:53:10 +0000
| Newsgroups | gmane.network.openvpn.devel |
|---|---|
| Message-ID | <8297e19a8ae0678ac2aa678f12c23bee71e66386-EmailReviewComments-HTML@gerrit.openvpn.net> |
--===============2059519310598756911== Content-Transfer-Encoding: 8bit Content-Disposition: inline Content-Type: multipart/alternative; boundary="vDZUBuH5pdQ="; charset=UTF-8 --vDZUBuH5pdQ= Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Attention is currently required from: cron2=2E plaisthos has posted commen= ts on this change by cron2=2E ( http://gerrit=2Eopenvpn=2Enet/c/openvpn/+/1= 830?usp=3Demail ) Change subject: SIGHUP-restart instance on UDP write err= or EADDRNOTAVAIL =2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E= =2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E= =2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E= =2E Patch Set 2: (1 comment) Commit Message: http://gerrit=2Eopenvpn= =2Enet/c/openvpn/+/1830/comment/0ef016e2_d2031880?usp=3Demail : PS2, Line 1= 7: not - so we can notice, and restart the instance=2E A full SIGHUP resta= rt I think SIGHUP is a bit too much here since it reloads the config and ot= her things=2E I think SIGUSR1 should be enough=2E If we need to remove rout= es/close tun we should have some flag/extended SIGURS1=2E -- To view, v= isit http://gerrit=2Eopenvpn=2Enet/c/openvpn/+/1830?usp=3Demail To unsubscr= ibe, or for help writing mail filters, visit http://gerrit=2Eopenvpn=2Enet/= settings?usp=3Demail Gerrit-MessageType: comment Gerrit-Project: openvpn G= errit-Branch: master Gerrit-Change-Id: I96b5ea3542df77b1be17079874fb9e4970b= 539ed Gerrit-Change-Number: 1830 Gerrit-PatchSet: 2 Gerrit-Owner: cron2 <ge= rt@greenie=2Emuc=2Ede> Gerrit-Reviewer: ordex <antonio@mandelbit=2Ecom> Ger= rit-Reviewer: plaisthos <arne-openvpn@rfc2549=2Eorg> Gerrit-CC: openvpn-dev= el <openvpn-devel@lists=2Esourceforge=2Enet> Gerrit-Attention: cron2 <gert@= greenie=2Emuc=2Ede> Gerrit-Comment-Date: Mon, 03 Aug 2026 10:53:10 +0000 Ge= rrit-HasComments: Yes Gerrit-Has-Labels: No --vDZUBuH5pdQ= Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable <!DOCTYPE html><html><head><style></style></head><body><p> Attention is cur= rently required from: cron2=2E </p> <p><a href=3D"http://gerrit=2Eopenvpn= =2Enet/c/openvpn/+/1830?usp=3Demail">View Change</a></p><p>1 comment:</p><u= l style=3D"list-style: none; padding: 0;"><li style=3D"margin: 0; padding: = 0;"><p>Commit Message:</p><ul style=3D"list-style: none; padding: 0;"><li s= tyle=3D"margin: 0; padding: 0 0 0 16px;"><p style=3D"margin-bottom: 4px;"><= a href=3D"http://gerrit=2Eopenvpn=2Enet/c/openvpn/+/1830/comment/0ef016e2_d= 2031880?usp=3Demail">Patch Set #2, Line 17:</a> <code style=3D"font-family:= monospace,monospace">not - so we can notice, and restart the instance=2E A= full SIGHUP restart</code></p><p class=3D"blocks" style=3D"white-space: pr= e-wrap; word-wrap: break-word;">I think SIGHUP is a bit too much here since= it reloads the config and other things=2E I think SIGUSR1 should be enough= =2E If we need to remove routes/close tun we should have some flag/extended= SIGURS1=2E</p></li></ul></li></ul><p>To view, visit <a href=3D"http://gerr= it=2Eopenvpn=2Enet/c/openvpn/+/1830?usp=3Demail">change 1830</a>=2E To unsu= bscribe, or for help writing mail filters, visit <a href=3D"http://gerrit= =2Eopenvpn=2Enet/settings?usp=3Demail">settings</a>=2E</p><div itemscope it= emtype=3D"http://schema=2Eorg/EmailMessage"><div itemscope itemprop=3D"acti= on" itemtype=3D"http://schema=2Eorg/ViewAction"><link itemprop=3D"url" href= =3D"http://gerrit=2Eopenvpn=2Enet/c/openvpn/+/1830?usp=3Demail"/><meta item= prop=3D"name" content=3D"View Change"/></div></div> <div style=3D"display:= none"> Gerrit-MessageType: comment </div> <div style=3D"display:none"> Gerr= it-Project: openvpn </div> <div style=3D"display:none"> Gerrit-Branch: mast= er </div> <div style=3D"display:none"> Gerrit-Change-Id: I96b5ea3542df77b1b= e17079874fb9e4970b539ed </div> <div style=3D"display:none"> Gerrit-Change-N= umber: 1830 </div> <div style=3D"display:none"> Gerrit-PatchSet: 2 </div> <= div style=3D"display:none"> Gerrit-Owner: cron2 <gert@greenie=2Emuc=2Ede= > </div> <div style=3D"display:none"> Gerrit-Reviewer: ordex <antonio= @mandelbit=2Ecom> </div> <div style=3D"display:none"> Gerrit-Reviewer: p= laisthos <arne-openvpn@rfc2549=2Eorg> </div> <div style=3D"display:no= ne"> Gerrit-CC: openvpn-devel <openvpn-devel@lists=2Esourceforge=2Enet&g= t; </div> <div style=3D"display:none"> Gerrit-Attention: cron2 <gert@gre= enie=2Emuc=2Ede> </div> <div style=3D"display:none"> Gerrit-Comment-Date= : Mon, 03 Aug 2026 10:53:10 +0000 </div> <div style=3D"display:none"> Gerri= t-HasComments: Yes </div> <div style=3D"display:none"> Gerrit-Has-Labels: N= o </div> </body></html> --vDZUBuH5pdQ=-- --===============2059519310598756911== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline --===============2059519310598756911== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel --===============2059519310598756911==--