SuppSandroB / sandrop

Automatically exported from code.google.com/p/sandrop
226 stars 62 forks source link

Unable to change script after setting it #128

Open HYBRID-BEING opened 7 years ago

HYBRID-BEING commented 7 years ago

After i've set pac-script, i can no longer change it or test it - each time i try to access this screen i get thrown out to settings. I can delete WiFi networks and set script once again, but there is nothing i can do about "not listed" networks.

SuppSandroB commented 7 years ago

Is there any errors in device logcat? You should also set script in "not listed" one. Will retest it.

HYBRID-BEING commented 7 years ago

2017-02-13-14-53-37.txt Here's a filtered log after trying to access script settings.

Script is already set in "not listed networks" and i no longer can change it.

SuppSandroB commented 7 years ago

It just crash app. Can you send content of script? Will try to reproduce problem. What kind of device do you have?

HYBRID-BEING commented 7 years ago

Ah, it just looked like i was "ejected" to settings tab (or WiFi networks list, if i tried to change script for specific network). Here is the script https://antizapret.prostovpn.org/proxy.pac My device is ASUS ZenFone Selfie, running Android 6.0.1.

HYBRID-BEING commented 7 years ago

So, did script cleared up anything?

SuppSandroB commented 7 years ago

Yes. Will fix it asap. Passing huge script content through activity intents is not proper solution :).

SuppSandroB commented 7 years ago

New version is on Google Play 113. It should fix this problem. But still freezes for a while before showing content of script. Let me know if it works for you.

HYBRID-BEING commented 6 years ago

Finally got motivated to try set up Drony again. Your previous fix worked but now Drony crashes on testing script (same one but bigger), so i cannot update it.

SuppSandroB commented 6 years ago

Looks like this split function is creating to much objects. Not sure if I can change this limit. Looks like hard-coded. https://stackoverflow.com/questions/13684232/freeing-jni-array?noredirect=1&lq=1 Can this script be changed to not do this? Or I must look at ducktape to fix it. https://github.com/square/duktape-android

HYBRID-BEING commented 6 years ago

Well, i guess that was to be expected. The script compiles web and IP addresses blocked in Russia, and there seem to be no end to expansion of the list. So rather than stripping unnecessary entries it'd be easier to just compile my own (really don't wanna though, haha) script.

Although i didn't quite catch how much entries Drony can handle. I'm pretty sure it was more than 512 last time it worked, though.