ppp icon indicating copy to clipboard operation
ppp copied to clipboard

CHAP Response add retransmission.

Open hanfezh opened this issue 11 years ago • 5 comments

CHAP Response add retransmission: transmited 10 times utils received SUCCESS or FAILURE, in 3 seconds interval.

hanfezh avatar Sep 01 '14 09:09 hanfezh

@nomis: What do you think about this PR?

Neustradamus avatar Nov 08 '20 03:11 Neustradamus

I've explained why retransmitting is a bad idea here: https://github.com/paulusmack/ppp/issues/87#issuecomment-595794742

This is a challenge/response authentication method and it is normal for the entire PPP session to be transparently transferred to a different LNS.

nomis avatar Nov 08 '20 08:11 nomis

@hanfezh: What do you think about @nomis comment?

At the same time, can you rebase and squash your commits and sign-off please?

Neustradamus avatar Dec 30 '20 04:12 Neustradamus

I've explained why retransmitting is a bad idea here: #87 (comment)

This is a challenge/response authentication method and it is normal for the entire PPP session to be transparently transferred to a different LNS.

RFC1334 says "the authenticator MUST allow repeated Response packets after completing the Authentication phase". To be sure, the peer could be buggy, and we can provide ways to work around that, but we don't have to assume that the peer is buggy.

paulusmack avatar Dec 31 '20 06:12 paulusmack

@hanfezh: What do you think about @nomis and @paulusmack comments?

Neustradamus avatar Sep 20 '21 14:09 Neustradamus

@hanfezh: Soon two years after my latest comment here.

What do you think?

Thanks in advance.

Neustradamus avatar Aug 24 '23 03:08 Neustradamus

@hanfezh: Soon two years after my latest comment here.

What do you think?

Thanks in advance.

Hi @Neustradamus I think we can close this PR, since it's about 9 years and no one else has feedback.

hanfezh avatar Aug 24 '23 07:08 hanfezh