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

Added a parser for SSL Proxy #51

Closed
wants to merge 12 commits into from
Closed

Added a parser for SSL Proxy #51

wants to merge 12 commits into from

Conversation

la55u
Copy link
Contributor

@la55u la55u commented Jul 26, 2018

added a parser + tests for sslproxy provider

p.s.: how do I 'hide' all these previous commits when creating a PR? I mean only the last two are new. thank you, sorry for being noob

la55u added 12 commits July 4, 2018 16:05
Proxies can now be filtered with an optional parameter in the RequestProxy constructor.
For this the parsers have to set the protocol for each proxy object they build.
Protocols are stored in the ProxyObject.Protocol enum.
PremProxy: parsing was broken because the port number was 'encrypted' and no longer stored in CSS.
They are now obtained from a javascript file that holds a function to the key-port pairs.
The default way the program gets user agents is now from an online, up-to-date database
with the help of the fake-useragent package. Reading useragents from a custom local
file is still available as a parameter to the UserAgentManager class. Solves pgaref#28.
@coveralls
Copy link

coveralls commented Jul 26, 2018

Coverage Status

Coverage increased (+1.3%) to 63.636% when pulling 25a7e42 on la55u:master into 5322364 on pgaref:master.

@pgaref
Copy link
Owner

pgaref commented Jul 28, 2018

Hey @la55u !
Thanks for the PR, good stuff!

To avoid the duplicate commits you should update your master branch which diverged from the original. https://stackoverflow.com/questions/1443210/updating-a-local-repository-with-changes-from-a-github-repository

Once you have done that you can port your changes and create a new clean PR.
Other than that the changes LGTM

@pgaref pgaref mentioned this pull request Oct 19, 2019
@pgaref pgaref closed this Oct 19, 2019
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

3 participants