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

Add stats counter and improvement tests async and add Message-Authenticator support #86

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

geaaru
Copy link
Contributor

@geaaru geaaru commented Jul 6, 2018

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

@geaaru
Copy link
Contributor Author

geaaru commented Jul 6, 2018

Wait, I missing some section of code with rebase of the master, i will update this soon.

@geaaru
Copy link
Contributor Author

geaaru commented Jul 6, 2018

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.

@GIC-de
Copy link
Collaborator

GIC-de commented Jul 20, 2018

Please rebase from master to solve the conflicts!!!

@geaaru
Copy link
Contributor Author

geaaru commented Aug 20, 2018

Can you merge pr now or do you think that is needed some more changes ?
I will try to integrate support to Status-Server message in a separated pr.

@geaaru geaaru force-pushed the master branch 2 times, most recently from b3f114b to a9bda29 Compare September 4, 2018 15:55
@geaaru geaaru changed the title Add stats counter and improvement tests async Add stats counter and improvement tests async and add Message-Authenticator support Sep 5, 2018
GIC-de added a commit that referenced this pull request Oct 19, 2019
@geaaru
Copy link
Contributor Author

geaaru commented Nov 1, 2019

Please, check my code with Message-Authenticator support and fix for CHAP authentication.

It seems that

chr(chapid).encode('utf-8')

doesn't work correctly when int > 128. I see this with radclient test.

I think that it's ready to merge. Let me know. Thanks

@GIC-de GIC-de force-pushed the master branch 2 times, most recently from f028133 to 5300091 Compare July 13, 2022 12:23
@GIC-de
Copy link
Collaborator

GIC-de commented Jul 24, 2024

Please rebase and resolve conflicts.

@geaaru
Copy link
Contributor Author

geaaru commented Jul 29, 2024

hey, sorry a bit busy at this moment but I will try to fix the conflicts in the next months. sorry for the delay.

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.

2 participants