-
Notifications
You must be signed in to change notification settings - Fork 5
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
Access limit by user/pass #4
Comments
sorry for my delay. oh ~ it's a rare use-case. Only very few users need auth . i think all you want to do in the handshake can place in the it init when a new client connect come , at the handshake from Socks5BalancerAsio/src/TcpRelayServer.cpp Line 150 in b58f7d1
it check the first 3 byte from client , and check it is a normal socks5 connect or a http connect : Socks5BalancerAsio/src/FirstPackAnalyzer.cpp Lines 148 to 223 in b58f7d1
(A) if it's a socks5 header (B) if it's seems like a http header , call the
if you want impl Auth in CASE (2) if you want impl Auth in CASE (1) |
OR, because impl the CASE (1)(2) have duplicate place. so we can impl they two. split (A) to do the socks5 handshake with upstream and downstream, so we can make a auth with server and client in socks5-socks5 mode. |
Oh this seems more complicated than I expected. Let me revisit this later. For more information on my use-case, I need client to be able to connect to Socks5BalancerAsio, only with a valid user/pass. And thanks for sharing Socks5BalancerAsio, I'm using it daily and extensively, has been a lot of help to me. |
Ok~ thanks for your use Socks5BalancerAsio . i will impl it later . please wait some times. |
Hi~ the auth impl now. about more information and how to enable it, please see the README now the AND, the new -- the v1.3 is first auth support version, |
Thanks so much! done fast! |
I built the version with the new feature (had to install a more recent boost)
|
oh, when i review the code i found that , the this issue only effect case (1) the case (2) |
Hi~
. i create the github auto build action, to make the pre-build binary. if you need deploy multi server, maybe you can use the auto build binary in github action OR in release like this that the binay file copy from github action. |
Hi~ the Auth Info Statistics be added to Web Monitor now. Release AutoBuild 2 i not sure the i18n work or not. you can modifiy js to force use en-US . Socks5BalancerAsio/html/boot.js Lines 165 to 176 in a6e830c
change it to : // moment.locale('zh-cn');
var getI18nTable = () => {
// english
window.i18nTable = window.i18n.enUS;
};
getI18nTable(); wait your feedback ~ |
Hi~ the auth support full impl now, add the i18n selector, and add the ping monitor. you can download the pre-build binary from Release page . i will close this issue, if you have any problem, please report again, thank you~ |
Thank you! |
Is access limit by user/pass available out of the box?
If not, do you think it'll be easy to add?
Can you guide me in implementing it myself, if you don't have time yourself?
The text was updated successfully, but these errors were encountered: