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.

Sign up for GitHub

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

Jump to bottom

MicroWebCli, line 260. SOCKS5 support #2

Open
mansour1991 opened this issue Feb 22, 2019 · 2 comments
Open

MicroWebCli, line 260. SOCKS5 support #2

mansour1991 opened this issue Feb 22, 2019 · 2 comments

Comments

@mansour1991
Copy link

Hello,

The BND.ADDR and The BND.PORT that is returned from the proxy server is not used on line 260.

cli.read(l + 2)

According to rfc1928 on top of page 6, these are the server bound address and the server bound port.

thanks
@mansour1991 mansour1991 changed the title MicroWebCli, line 260 MicroWebCli, line 260. SOCKS5 support Feb 22, 2019
@jczic
Copy link
Owner

jczic commented Feb 22, 2019

Yes I know that but I really added the sock5 part quickly for a little compatibility on esp32 on a need 👎

@uxhamby
Copy link

uxhamby commented Aug 3, 2020

Is there any documentation on the use of 'wCli.Socks5Addr' ? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jczic @uxhamby @mansour1991