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

Allow to connect to Facebook through TOR #403

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

Conversation

rezemika
Copy link

@rezemika rezemika commented Mar 1, 2019

Hi!

Here is an humble proposal of a simple modification that allows to connect to Facebook through TOR, as if a user would use Tor Browser to facebookcorewwwi.onion. I wrote this for a personal project, so here is a PR if you think it can be useful. :)

@madsmtm
Copy link
Member

madsmtm commented Mar 6, 2019

Thanks for the PR! However, I probably won't merge it, I'm trying to take a more conservative stand on non-essential functionality, at least while I'm doing a lot of code cleanup 😊. It should be possible to provide this functionality by extending Client and ReqUrl manually, though. If you provided that as an example instead, then I'll probably redecide 😉

@rezemika
Copy link
Author

Well, I'm sorry for the late of my answer.

I'm not sure it's possible to provide this as an example with changing anything in the base code, because it needs to change Client attributes before the end of its __init__ method. For example, it's not possible to change the self._session attribute, but it's mandatory to set the new proxies.

Else, it's possible to create a class which inherit from Client and overrides its __init__ method, but it won't work anymore if you plan to rewrite it and change the way it works in your future code cleanup.

@madsmtm
Copy link
Member

madsmtm commented Mar 17, 2019

Yeah, I can see that. I think I'll keep this PR open until we reach v2, where we can then discuss how this can be implemented (the code in __init__ will probably be restructured in such a fasion that this change will be possible)

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.

2 participants