Skip to content
This repository has been archived by the owner on Mar 3, 2020. It is now read-only.

Socks5 support #709

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

Conversation

addicted2sounds
Copy link

simple modifications for socks5 support

@mhoran
Copy link
Collaborator

mhoran commented Jan 22, 2015

Thanks for the pull request! It'd be great if you could provide tests for this feature. Check out spec/browser_spec.rb to see how we test the HTTP proxy.

@addicted2sounds
Copy link
Author

I'm a bit confused about how to run existing tests for webkit. I couln't start any tests on my local machine, so I'm not sure how to write tests :( right now it says that there's no webkit-server file in repo.

@jferris
Copy link
Contributor

jferris commented Jan 22, 2015

If you run rake, it will build the server before trying to run all the tests.

If you'd like to just run individual tests, you can run rake build first.

@jferris
Copy link
Contributor

jferris commented Apr 10, 2015

@addicted2sounds I just wanted to check in and see if you're still interested in writing tests for this.

@addicted2sounds
Copy link
Author

Hi, yes, I'm interested but since I have no much time, I couldn't do them. I need to read documentation of how socks server responds and that inhibits me

@addicted2sounds
Copy link
Author

It looks I have no idea how to emulate socks proxy behaviour in tests. Could anyone help me?

end
end
end
=======

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unexpected token tEQQ
(Using Ruby 2.3 parser; configure using TargetRubyVersion parameter, under AllCops)

port: "0",
user: "",
pass: "",
type: "http"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put a comma after the last item of a multiline hash.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants