Skip to content

Changed the default ports to avoid conflict with The Onion Router daemon#1

Open
fvcr wants to merge 1 commit intoscuty2000:masterfrom
fvcr:master
Open

Changed the default ports to avoid conflict with The Onion Router daemon#1
fvcr wants to merge 1 commit intoscuty2000:masterfrom
fvcr:master

Conversation

@fvcr
Copy link

@fvcr fvcr commented Oct 14, 2021

I was unable to run TorRequest using the same Tor process, alternatively,
I thought it might be feasible to change the default ports to avoid conflicts.

Were you able to successfully run it with the tor daemon running?

I would like to know your opinion and thanks for continuing the project.

@scuty2000
Copy link
Owner

scuty2000 commented Oct 30, 2021

Thank you for contributing! I'll have a look in a few days, i'm busy at work for now.

@scuty2000
Copy link
Owner

Hi, I completely forgot to reply you. I assume that it would be better to make the ports configurable directly in the constructor instead of editing the code to edit them. I'm gonna take a look at it this week (I hope!)

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