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

Proxy Port Fixes #715

Merged
merged 9 commits into from
Jul 26, 2022
Merged

Proxy Port Fixes #715

merged 9 commits into from
Jul 26, 2022

Conversation

bitmold
Copy link
Collaborator

@bitmold bitmold commented Jul 20, 2022

This branch is dependent on guardianproject/tor-android#84 being released in tor-android and should not be merged until that is released
Fixes: #708 and #709

@bitmold
Copy link
Collaborator Author

bitmold commented Jul 20, 2022

Also fixes #707

@bitmold bitmold force-pushed the proxy_port_fixes branch from 81a5c1d to b1f33b6 Compare July 20, 2022 22:27
extraLines.append("SafeSocks 0").append('\n');
extraLines.append("TestSocks 0").append('\n');
} else {
extraLines.append("SOCKSPort 0.0.0.0:").append(socksPortPref).append("\n");
Copy link
Contributor

@SkewedZeppelin SkewedZeppelin Jul 20, 2022

Choose a reason for hiding this comment

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

should isolate/ipv6 be added here too? 51216a4

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes, thank you. added

@syphyr
Copy link
Contributor

syphyr commented Jul 21, 2022

5ba7ce9 has a missing symbol

@syphyr
Copy link
Contributor

syphyr commented Jul 21, 2022

It looks like R.id.tvSnowflake is undefined

@syphyr
Copy link
Contributor

syphyr commented Jul 21, 2022

This fixes the problem: syphyr@ef7d989

@bitmold
Copy link
Collaborator Author

bitmold commented Jul 21, 2022

Thanks @syphyr , I just forgot to commit the XML file. I'll throw it up next time I'm online

@syphyr
Copy link
Contributor

syphyr commented Jul 21, 2022

One other thing I noticed is that the version string is only showing up for obfs4 in "About" screen.

@bitmold
Copy link
Collaborator Author

bitmold commented Jul 21, 2022

Odd, I was seeing both versions. Will revisit when I commit the missing XML file ...

@bitmold bitmold force-pushed the proxy_port_fixes branch from 15cbe18 to 13217fa Compare July 24, 2022 18:25
@SkewedZeppelin
Copy link
Contributor

SkewedZeppelin commented Jul 25, 2022

eh, guardianproject/tor-android#84 was merged and released?

weren't these supposed to be together?

@n8fr8 n8fr8 merged commit c3aab3c into master Jul 26, 2022
@bitmold bitmold deleted the proxy_port_fixes branch August 3, 2022 15:23
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.

Orbot ignores Isolate destination addresses
4 participants