-
Notifications
You must be signed in to change notification settings - Fork 190
Add stats counter and improvement tests async and add Message-Authenticator support #86
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
base: master
Are you sure you want to change the base?
Conversation
|
Wait, I missing some section of code with rebase of the master, i will update this soon. |
|
Sorry, I create wrong VerifyAuthPacket on previous pr and I fix documentation for every verify methods (it was inverted). I tested it with freeradius and it seems works fine. I'm not sure that current async server support correctly Status-Server feature, I will try to test and integrate eventually some fix in the next weeks. |
|
Please rebase from master to solve the conflicts!!! |
|
Can you merge pr now or do you think that is needed some more changes ? |
b3f114b to
a9bda29
Compare
* Packet class now extend OrderedDict to maintain correct AVP order and HMAC/MD5 check fo Message-Authenticator * Fix client_async for Accounting, Coa
|
Please, check my code with Message-Authenticator support and fix for CHAP authentication. It seems that doesn't work correctly when int > 128. I see this with I think that it's ready to merge. Let me know. Thanks |
f028133 to
5300091
Compare
|
Please rebase and resolve conflicts. |
|
hey, sorry a bit busy at this moment but I will try to fix the conflicts in the next months. sorry for the delay. |
Some statistics enhancements for measure performance. On local machine with async server reach 600p/s requests without lost packets.
Can you bump a new release ?
Thanks in advance