Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PAP authentication added, do_build_payload changed to build_payload #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

denysaleksandrov
Copy link

I added PAP Authentication and changed "do_build_payload" function to "build_payload" as the initail one was changed in scapy/packet.py.
I tested PAP, CHAP and both authentication value specified at the same time.
In order to add authentication I use ">>> p.authentication = ['chap', 'pap', 'both']":

p.authentication = ['chap']":

  1. if only CHAP is specified and PPPoE server supports only CHAP or both CHAP and PAP
    then CHAP would be used

p.authentication = ['pap']":
2) if only PAP is specified and PPPoE server supports only PAP or both CHAP and PAP
then CHAP would be used

p.authentication = ['chap', 'pap']
or
p.authentication = ['both']
3) if PPPoE server sent CHAP challange, CHAP auth would be used
if PPPoE server didn't send anything, PAP auth would be used

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant