yalu102 icon indicating copy to clipboard operation
yalu102 copied to clipboard

Logic refactor & URL Schemes / 3D Touch

Open nullpixel opened this issue 8 years ago • 20 comments

  • Made already jailbroken checking a function
  • Removed didReceiveMemoryWarning
  • Added
else if (strstr(u.version, "MarijuanARM")) {
        printf("Already jailbroken\n");
    }

in offsets.c to correctly log the jailbroken state.

  • Added a URL scheme to jailbreak: yalu://break
  • Added a 3D touch action to jailbreak
  • Added a jailbreaking button state
  • Hugely refactored logic
  • App now supports the simulator for testing

Bigger view controller refactor coming soon.

nullpixel avatar Jan 28 '17 12:01 nullpixel

The app can now be built for and run in the simulator. Upon pressing "go", the user is greeted with the following screen:

Simulator jailbreak alert

AppleBetas avatar Jan 28 '17 17:01 AppleBetas

^ We're currently using it to manipulate bools, and test 3D touch

nullpixel avatar Jan 28 '17 17:01 nullpixel

Okay, this should be good to merge now!

nullpixel avatar Jan 28 '17 18:01 nullpixel

I approve this. 3D touching works! Just gave it a go

TheNightRider12 avatar Jan 29 '17 00:01 TheNightRider12

@kpwn I rebased

nullpixel avatar Jan 29 '17 03:01 nullpixel

@parrotgeek1, that's exactly what it does.

AppleBetas avatar Jan 29 '17 05:01 AppleBetas

it works amazing :) I love it. 3D touch is easier to me

TheNightRider12 avatar Jan 29 '17 05:01 TheNightRider12

@Mila432 What part of the code are you referring to?

nullpixel avatar Jan 29 '17 18:01 nullpixel

@nullpixel1 https://github.com/kpwn/yalu102/pull/141/commits/d02d694badd041246e0b33a1653ac862db66fe74 no idea why GitHub moved the comment here

Mila432 avatar Jan 29 '17 18:01 Mila432

@Mila432 ohhhh

Look at the file it's in: it's my own jailbreak check, not the offsets.

To shed more light on it, ==0 was a debug statement I added to always make that return true for testing, but I forgot to remove it from the vc before committing.

nullpixel avatar Jan 29 '17 18:01 nullpixel

@nullpixel1 ye old code was c= a == b , new code c = a

Mila432 avatar Jan 29 '17 18:01 Mila432

yeah, it was a debug statement @Mila432

nullpixel avatar Jan 29 '17 18:01 nullpixel

@kpwn this PR now updates readme for new jailbreak, and should clarify some support questions.

nullpixel avatar Jan 29 '17 18:01 nullpixel

I am not OK with a URI scheme to initiate a jailbreak. Sounds sketch.

kpwn avatar Jan 29 '17 18:01 kpwn

@kpwn The user has to press ok to do it when launched via a URL scheme. It's mainly there for things such as workflow.

nullpixel avatar Jan 29 '17 18:01 nullpixel

@kpwn Sorry, forgot to commit the alert view, can you re-review this now?

nullpixel avatar Jan 29 '17 18:01 nullpixel

Rebased

nullpixel avatar Jan 29 '17 21:01 nullpixel

Resolved conflicts

AppleBetas avatar Jan 29 '17 23:01 AppleBetas

Resolved merge conflict

nullpixel avatar Jan 31 '17 07:01 nullpixel

@kpwn

Did you change your mind about URL schemes or should I factor them out?

nullpixel avatar Feb 03 '17 17:02 nullpixel