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

Enable custom Shadowsocks port selection on entry servers in multihop #7263

Conversation

rablador
Copy link
Contributor

@rablador rablador commented Nov 28, 2024

Steps to reproduce

  • Select a relay that has shadowsocks_extra_addr_in enabled
  • Connect to it in single hop mode
  • Select a custom shadowsocks obfuscation port to validate that it supports any ports in the 1-65535 range
  • Disconnect
  • Turn on multihop
  • Select the aforementioned server as an entry point for multihop
  • Select any other server as an exit hop
  • Tap on "Secure Connection"

Expected Result

The app should just connect using Shadowsocks obfuscation on the entry server.

Actual result

The app enters the blocked state, thinking that the relay constraints are not valid.


This change is Reviewable

@rablador rablador added bug iOS Issues related to iOS labels Nov 28, 2024
@rablador rablador self-assigned this Nov 28, 2024
Copy link

linear bot commented Nov 28, 2024

@rablador rablador force-pushed the enable-custom-shadowsocks-port-selection-on-entry-servers-in-ios-955 branch from d8b714e to 1478ff6 Compare November 28, 2024 21:01
buggmagnet
buggmagnet previously approved these changes Dec 2, 2024
Copy link
Contributor

@buggmagnet buggmagnet left a comment

Choose a reason for hiding this comment

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

Reviewed 10 of 10 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

mojganii
mojganii previously approved these changes Dec 2, 2024
Copy link
Collaborator

@mojganii mojganii left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@rablador rablador force-pushed the enable-custom-shadowsocks-port-selection-on-entry-servers-in-ios-955 branch from 1478ff6 to 23ebe03 Compare December 3, 2024 07:30
@rablador rablador dismissed stale reviews from buggmagnet and mojganii via aa36f07 December 3, 2024 07:32
@rablador rablador force-pushed the enable-custom-shadowsocks-port-selection-on-entry-servers-in-ios-955 branch from 23ebe03 to aa36f07 Compare December 3, 2024 07:32
Copy link
Collaborator

@mojganii mojganii left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: 9 of 12 files reviewed, all discussions resolved (waiting on @buggmagnet)

Copy link
Contributor

@buggmagnet buggmagnet left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 3 of 3 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@rablador rablador force-pushed the enable-custom-shadowsocks-port-selection-on-entry-servers-in-ios-955 branch from aa36f07 to a9a0189 Compare December 3, 2024 09:49
@rablador rablador merged commit 8e49038 into main Dec 3, 2024
11 checks passed
@rablador rablador deleted the enable-custom-shadowsocks-port-selection-on-entry-servers-in-ios-955 branch December 3, 2024 09:51
@rablador rablador restored the enable-custom-shadowsocks-port-selection-on-entry-servers-in-ios-955 branch December 3, 2024 09:53
Copy link

github-actions bot commented Dec 3, 2024

🚨 End to end tests failed. Please check the failed workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug iOS Issues related to iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants