-
Notifications
You must be signed in to change notification settings - Fork 927
Description
Is your feature request related to a problem? Please describe.
To make android sslpinning disable
work for this app I had to change this line:
objection/agent/src/android/pinning.ts
Line 144 in c2121af
const CertificatePinnerCheckOkHttp = certificatePinner.check$okhttp.overload("java.lang.String", "u15"); |
to:
const CertificatePinnerCheckOkHttp = certificatePinner.check$okhttp.overload("java.lang.String", "kotlin.jvm.functions.Function0");
Describe the solution you'd like
I suspect other non-obfuscated Kotlin apps will require a similar change.
However there are already two big function blocks just for okhttp, so adding a third one doesn't feel right.
So this issue is somewhat a meta question: to what extend to you want to include/support all these different function signatures? And how? By replicating a lot of code, or introducing another level of abstraction somewhere?
Describe alternatives you've considered
Add a third okHttp3CertificatePinnerCheckXYZ
function.
Additional context
n/a