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

5.1.2 doesn't work with v2ray-plugin (with default plugin configuration) #2562

Closed
5 of 17 tasks
iKirby opened this issue Jul 16, 2020 · 4 comments
Closed
5 of 17 tasks
Labels

Comments

@iKirby
Copy link

iKirby commented Jul 16, 2020

Describe the bug
Version 5.1.2 doesn't work with v2ray-plugin. Same configuration works on 5.0.6, and also works on Windows version.

To Reproduce

  1. Setup a profile with v2ray-plugin
  2. Connect
  3. Do the connectivity test
  4. See error message "Unable to resolve host", and there is no internet connection
  5. Check logcat, there are messages saying that libsslocal failed to connect to plugin

Expected behavior
Shadowsocks works well with v2ray-plugin.

Smartphone (please complete the following information):

  • Android version: 10 (QQ3A.200605.001, LineageOS 17.1-20200704-NIGHTLY-sailfish)
  • Device: Google Pixel
  • Version: Pre-release 5.1.2
  • Last version that did not exhibit the issue: Latest release 5.0.6

Configuration

  • IPv4 server address
  • IPv6 server address
  • Client IPv4 availability
  • Client IPv6 availability
  • Encrypt method:
  • Route
    • All
    • Bypass LAN
    • Bypass China
    • Bypass LAN & China
    • GFW List
    • China List
    • Custom rules
  • IPv6 route
  • Apps VPN mode
    • Bypass mode
  • Remote DNS: dns.google
  • DNS over UDP
  • Plugin configuration (if applicable):
  • Auto Connect
  • TCP Fast Open

Additional context
"All" route mode doesn't work either.

Logcat

07-16 09:02:16.959 20300 20300 I GuardedProcessPool: start process: /data/app/com.github.shadowsocks-_m8azGdhDsWKnm9NXdEMuw==/lib/arm64/libtun2socks.so --netif-ipaddr 172.19.0.2 --socks-server-addr 127.0.0.1:1080 --tunmtu 1500 --sock-path sock_path --dnsgw 127.0.0.1:5450 --loglevel warning --enable-udprelay
...
07-16 09:02:17.136 20300 21113 E libsslocal: 2020-07-16T09:02:17.135+08:00 INFO  shadowsocks 1.8.12
07-16 09:02:17.137   648 18702 I netd    : networkAddRoute(129, "tun0", "198.64.0.0/10", "") <0.25ms>
07-16 09:02:17.138   648 18702 I netd    : networkAddRoute(129, "tun0", "198.128.0.0/9", "") <0.24ms>
07-16 09:02:17.139   648 18702 I netd    : networkAddRoute(129, "tun0", "199.0.0.0/8", "") <0.28ms>
07-16 09:02:17.140   648 18702 I netd    : networkAddRoute(129, "tun0", "200.0.0.0/7", "") <0.26ms>
07-16 09:02:17.141 20300 21113 E libsslocal: 2020-07-16T09:02:17.140+08:00 INFO  started plugin "/data/app/com.github.shadowsocks.plugin.v2ray-0SL8ulrtXDhPUZ3ebP6z9Q==/lib/arm64/libv2ray.so" on 127.0.0.1:37103 <-> 173.82.186.78:465 (21127)
...
07-16 09:02:17.347 21127 21127 I v2ray   : V2Ray 4.23.2 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.14.3 android/arm64)
07-16 09:02:17.347 21127 21127 I v2ray   : A unified platform for anti-censorship.
07-16 09:02:17.350 21127 21127 W v2ray   : v2ray.com/core: V2Ray 4.23.2 started
...
07-16 09:02:22.808 20300 21113 E libsslocal: 2020-07-16T09:02:22.808+08:00 INFO  shadowsocks TCP listening on 127.0.0.1:1080
07-16 09:02:22.809 20300 21113 E libsslocal: 2020-07-16T09:02:22.809+08:00 INFO  shadowsocks DNS relay listening on tcp:127.0.0.1:5450
07-16 09:02:22.810 20300 21113 E libsslocal: 2020-07-16T09:02:22.809+08:00 INFO  shadowsocks DNS relay listening on udp:127.0.0.1:5450
07-16 09:02:24.984 20300 21113 E libsslocal: 2020-07-16T09:02:24.983+08:00 ERROR failed to connect 127.0.0.1:37103, retried 3 times, last_err: Connection refused (os error 111)
07-16 09:02:25.623   903  1216 E LightsService: Light requested not available on this device. 2
07-16 09:02:25.684   648 18702 I netd    : tetherGetStats() <4.90ms>
07-16 09:02:29.941 20300 21113 E libsslocal: 2020-07-16T09:02:29.940+08:00 ERROR failed to connect 127.0.0.1:37103, retried 3 times, last_err: Connection refused (os error 111)
07-16 09:02:34.947 20300 21113 E libsslocal: 2020-07-16T09:02:34.946+08:00 ERROR failed to connect 127.0.0.1:37103, retried 3 times, last_err: Connection refused (os error 111)

I thinks this should be the same as #2515 , but that has been closed due to inactivity. I'm sorry for my late reply on that issue. Logcat I captured before is at #2515 (comment) .

@iKirby iKirby added the bug label Jul 16, 2020
@madeye
Copy link
Contributor

madeye commented Jul 16, 2020

I cannot reproduce this locally.

From your log, v2ray looks killed at the very beginning. Have you installed any app that prevents background wake up?

@iKirby
Copy link
Author

iKirby commented Jul 16, 2020

I'm really sorry. It's my fault. Just checked all the configurations, and found that plugin configurations are not imported when scanning QR code (generated from Windows version). In the previous logcat I remember that I tested using a server without special plugin configuration, may be I messed it up. Now it works well, thanks for your reply.

@iKirby iKirby closed this as completed Jul 16, 2020
@iKirby
Copy link
Author

iKirby commented Jul 16, 2020

@madeye After more testing, I think I found the issue.

If I use v2ray-plugin without any configuration (with only one option "server"), and leave everything just default on v2ray-plugin for Android (plugin configuration "Not set"), it doesn't work on 5.1.2 but works on 5.0.6.

If I change the hostname to something other than "cloudfront.com", it works on 5.1.2.

If configuration is required in the new version, could you please add something to notify users that it needs to be configured manually?

@iKirby iKirby reopened this Jul 16, 2020
@iKirby iKirby changed the title 5.1.2 doesn't work with v2ray-plugin 5.1.2 doesn't work with v2ray-plugin (with default plugin configuration) Jul 16, 2020
@madeye
Copy link
Contributor

madeye commented Jul 16, 2020

@Mygod Looks like if the plugin options are not set, shadowsocks-rust will pass empty PLUGIN_OPTS to the plugin, which caused the issue here.

@madeye madeye closed this as completed in d2f60c5 Jul 16, 2020
Mygod added a commit that referenced this issue Jul 16, 2020
JackyAnn added a commit to JackyAnn/shadowsocks-android that referenced this issue Sep 22, 2022
* Allow user to keep data when uninstalling (shadowsocks#2506)

* Update README.md

* Update dependencies

* Add cargo clean task

* Enable LTO

* Speed up rebuild

* Update issue templates with labels

* Update dependencies

* Fix output name second attempt

* Update NDK

* Ensure cargoBuild is ran before mergeJniLibFolders

* Bump version

* Refine release build process

* Refine gradle files

* Add RUST_BACKTRACE

* Use rethrowAsSocketException

* Suppress write errors to protect_path

* Update shadowsocks-rust

* Bump version

* Improve accessibility

* Rename cipher plain to none

* Revert "Add RUST_BACKTRACE"

This reverts commit fdff88e.

ndk-stack should be used instead.

* Use cp.cloudflare.com for connectivity test

Credits: https://www.v2ex.com/t/656983#r_8748918

* Update dependencies

* Remote DNS setting is enabled unconditionally

* Show more information on conflicting plugins

Because one of you feckers just cannot learn to be decent.

* Add support for PTR queries

* Prevent querying PTR on custom Network

* Suppress network unspecified exceptions

* Handle IOException while reading

* Refine handling duplicate plugins

* Use any address as default DNS

* Suppress EACCES for ProtectWorker

* Clean up unused code

* Do not suppress IOException

* Disable UDP relay if plugin is enabled and no fallback

* Remove unnecessary isExperimental

* Update dependencies

* Fix unchecked cast

* Do not use WorkManager in device storage

* Remove unnecessary directBootAware overloading

* Suppress BadConfigurationProvider

* Disable RemoveWorkManagerInitializer lint for apps

* Update to Android 11 beta 1

* Request QUERY_ALL_PACKAGES for mobile

* Refine code style

* Fix ambiguous coroutineContext

* Downgrade coroutines

* Update dependencies

* Deprecate using Handler

* Fix shadowsocks#2546

* Only match exported plugins

* Update dependencies

* Update dependencies

* Migrate to ML kit for scanning QR code

Fixes shadowsocks#2548.

* Make scanner immersive

* Lock orientation to prevent camera recreation

* Fix missing Serializable declaration

* Add missing serialVersionUID

* Update dependencies

* Partially migrate to ActivityResultContracts

AlertDialogFragment will be migrated after the API goes stable.

* Add ActionBar to oss activity

* Add ActionBar to details activity

* Refine Scanner

* Bump version

* Update dependencies

* Suppress cancellation exceptions

* Skip processing if EOS is reached

* Refine PTR compat

* Fix shadowsocks#2557

* Fix shadowsocks#2562

* Update Android gradle

* Set VPN flag properly

Refine shadowsocks#2562.

* Drop support for Android Lollipop

* Update dependencies

* Remove UDP upstream DNS support. Fix shadowsocks#2564 shadowsocks#2518

* Update shadowsocks-rust

* Update dependencies

* Decouple main dependencies from plugin lib

* Simplify code

* Add missing type

* Update dependencies

* Deprecate old backup mechanism for Android 5-

* Update leanback theme to appcompat

* Use singleTask launchMode

* Fix build

* Bump version

* Update dependencies

* Update dependencies

* Bump plugin lib version to 2.0.0

* Add isV2 to PluginManager.InitResult

* Pass a value with the VPN option, if plugin version < 2.0

* Switch to __android_vpn

* Remove unnecessary parentheses

* Update shadowsocks-rust

* Bump version

* Clean up and bump version

* Revert camera-view back to alpha17

* Bump version

* Update dependencies

* Migrate away from deprecated APIs with core 1.3.0-alpha05

* Clean up code

* Refine code style

* Remove old ciphers. Fix shadowsocks#2621

* Enable single-threaded

* Update shadowsocks-rust to 1.8.23

* Use Parcelize for TrafficStats

* Fix platform insets on API 29-

* Fix shadowsocks#2623

* Update barcode-scanning

* Fix shadowsocks#2571

* Refine error message

* Refine shadowsocks#2571

* Limit open sockets to 256 in UDP association. Fix shadowsocks#2625

* Bump version

* Fix little problem of profile switching

* Update shadowsocks-rust to 1.9.0 (shadowsocks#2622)

* Fix shadowsocks#2638

* Bump version

* Switch to local UDP DNS resolver (shadowsocks#2635)

* Switch to the local UDP DNS resolver

* Update shadowsocks-rust

* Revert the rustup commands

* Fix shadowsocks#2642

* Fix the ByteBuffer allocation

* Update shadowsocks-rust

* Revert to local UDS resolver

Fix shadowsocks#2650

* Check deprecated ciphers (shadowsocks#2651)

* Bump version

* Fix shadowsocks#2301

* Bump version

* Remove the non-ietf chacha20 and salsa20 ciphers

* Fix shadowsocks#2665

* Update dependencies

* Fix deprecation of kotlin extensions

* Remove unused gcm work library for API 23+

* Use work-multiprocess

* Refine code style

* Update dependencies (shadowsocks#2672)

* Update dependencies

* Refine code style

* Bump gradle to 6.8.2

* Drop packet if out of buffer

* Refine error message

* Enlarge the buffer size of a UDP connection

* Bump version

* Ignore all exceptions whilst updating subscriptions

* Refine the JSON config file for ss-rust 1.10

* Refine the code style

* Update shadowsocks-rust to v1.10.3

* Fix shadowsocks#2679

* Update dependencies

Fixes shadowsocks#2699.

* Fix deprecation of adapterPosition

* Bump plugin to 2.0.1

* Migrate AlertDialogFragment to fragment result API

* Remove useless dependency update

* Migrate the rest to fragment result API

* Remove unused field

* Prevent crashing on shitty ROMs

* Add search tool for profiles (shadowsocks#2682)

* Suppress logging unsupported query type errors

* Ignore if connection was prematurely closed

* Use default udp_timeout=300 instead

Refine shadowsocks#2625.

* Mark underlyingNetwork as volatile

Attempts at addressing shadowsocks#2667.

* Fixed required targetFragments for preference

* Downgrade gradle plugin

* Update shadowsocks-rust and add back some ciphers

Fix shadowsocks#2705 and shadowsocks#2663.

* Update dependencies

* Check plugin properly

Fix shadowsocks#2667.

* Check crypto before init

* Make code style more Kotlin

* Update core and remove workaround

* Refine search to respect locale

* Double fixes touch target

* Resolve server name dynamically always (shadowsocks#2731)

Fixes shadowsocks#2722.

* Bump version

* Downgrade fragment to 1.3.2

Fixes shadowsocks#2733.

* Make plugin library depend on minimum version instead

* Enable Parallel GC

* Set useLegacyPackaging

* Update the maven publish plugin (shadowsocks#2734)

1. Remove the unnecessary custom URL
2. Replace jcenter with mavenCentral

* Handle illegal profiles properly

* Fix windows build failed shadowsocks#2666 shadowsocks#2711

* Allowed build under msys2 or cygwin

* Update dependencies

* Bring back semitransparent TV app via build variants (shadowsocks#2741)

This creates two variants for tv, freedom containing the original design, and google containing one matching Google's nonsense guideline. The former should be published on GitHub and elsewhere.

Revert "Revert "Revert "Revert "Revert "Make app fullscreen to match guidelines"""""

This reverts commit 0c67ac6.

* Fix typo

* Show full proxied apps mode to compensate for missing title

* Use MaterialAlertDialogBuilder

* Add progress indicator to ServiceButton

* Remove elevation from progress

* Only show progress for connecting

* Only show progress when connecting is taking too long

* Bump version

* Update dependencies

* Update sdk to S

* Use registerBestMatchingNetworkCallback

* Use OsConstants.ENONET

* Deal with new requirements

* Use main thread for default network callback on API 26+

* Remove unused code

* Fix duplicate authentication

* Update dependencies

* Bump version

* More stupid Android 6 bugs

* Update dependencies

* Downgrade leanback-preference

* Update to API 31

* Remove STORAGE permission on Android 10+

* Update dnsjava

* Ensure that redirects are always followed

Fixes shadowsocks#2786, shadowsocks#2791.

* Update dependencies

* Update to AGP 7.0.2

* Fix lint

* Update dependencies

* Fix shadowsocks#2803

* Remove extra file

* v5.2.6

* Revert "Show full proxied apps mode to compensate for missing title"

This reverts commit d21cf7b.

Fixes shadowsocks#2806.

* Add linkedin.com to gfwlist

* Fix Codacy badge

* Update dependencies

* Update .gitignore

ignore DS_Store

* Just to satisfy the obsessive-compulsive disorder :-)

* Update shadowsocks-rust

Update rust  dependency

* add new ciphers feature, and reorder cipher name

* Modify acl rules

* Update config.yml

try to fix up the rust build error

* enable armv8, neon feathures for hardware acceleration.

* Fix the missing springAnimator

* Update shadowsocks-rust to v1.15.0-alpha.5

* Bump version

* Update translations

Fixes shadowsocks#2867.

* Add German and Ukrainian translations

* Misc fixes

* Fix order

* Use system resolver

* Update dependencies

* Declare POST_NOTIFICATIONS

* Protect sensitive profile information when copied

* Add support for monochrome icons (not recommended)

* Declare supported languages

* Require authentication for closing service

* Support skipping animation in StatsBar

* Downgrade AGP

* Misc fixes

* Turn on fun switch

* Remove useless code

* Bump version

* Fix the compile error shadowsocks#2930 (shadowsocks#2935)

Make checking python version code compatible with windows and *nix

Redefine the python version detection code

Co-authored-by: Mygod <contact-github@mygod.be>
Co-authored-by: Mygod <contact-git@mygod.be>
Co-authored-by: Max Lv <max.c.lv@gmail.com>
Co-authored-by: cyber386 <62885475+cyber386@users.noreply.github.com>
Co-authored-by: Goooler <wangzongler@gmail.com>
Co-authored-by: SquallATF <squallatf@gmail.com>
Co-authored-by: dev4u <dev4u@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants