ppp icon indicating copy to clipboard operation
ppp copied to clipboard

Dial on demand does not trigger whith the "sync"-option

Open sthibaul opened this issue 4 years ago • 2 comments

From https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=75797

“ With the upstream "ppp-2.3.11", dial on demand does not trigger when the "sync"-option is enabled. [...] The reason is that when demand reads the frames from the loopback, in sync mode, it doesn't recognize any valid frame because these frames don't start with PPP_FLAG (I don't know why). ”

In Debian we are carrying the attached patch oedod.txt

but I don't know if this is still relevant

sthibaul avatar Dec 30 '20 11:12 sthibaul

Hmmm, dial-on-demand was never tested with the sync option, and in fact I don't know what devices would use that option.

To be accepted, the patch would need a sign-off and a description that explained the changes in the patch - in particular, what the large lump of code added to demand_rexmit() is doing.

paulusmack avatar Dec 31 '20 06:12 paulusmack

Yes, the patch seemed not that clear to me ("I don't know why" is not a good reason :) ), that's why I posted it here as an issue rather than a PR, so people can comment on this whether they need it.

sthibaul avatar Dec 31 '20 09:12 sthibaul