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

Adding Radius AVP Called-Station-Id #204

Merged
merged 3 commits into from Nov 27, 2016
Merged

Conversation

LegDog
Copy link

@LegDog LegDog commented Nov 26, 2015

If you are using RADIUS for authentication for multiple Virtual Hubs in SoftEther,, then it is nice to have Radius know which Hub is sending the auth request. As Softether VPN server sends a hard-coded NAS-Identifier attibute, and we may have many hubs using the same server IP address, the other suitable standard Radius attibute for this task is Called-Station-ID, for which the actual format of the information is site or application specific.

@dnobori
Copy link
Member

dnobori commented May 26, 2016

Your great patch is much appreciated. We are considering to apply your patch into the SoftEther VPN main tree.

SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch

You have two options which are described on the above policy.
Could you please choose either option 1 or 2, and specify it clearly on the reply?

@LegDog
Copy link
Author

LegDog commented Sep 19, 2016

Option 1, you are allowed to apply the same patch into PacketiX VPN if you like. Thanks.

@dnobori dnobori merged commit 034a213 into SoftEtherVPN:master Nov 27, 2016
@dnobori dnobori mentioned this pull request Nov 27, 2016
@dnobori
Copy link
Member

dnobori commented Nov 27, 2016

Thank you so much for your contribution to enrich the SoftEther VPN source code.

Your patch has been merged on the main source-tree of SoftEther VPN.

As a token of our gratitude, your GitHub username has been added on the AUTHORS.TXT file and on the header of the related source file.
Please see: https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT

Thanks again for your contribution.

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.

None yet

2 participants