mirror of
https://github.com/revanced/revanced-patches
synced 2024-09-12 17:26:45 +02:00
fix(Tumblr - Fix old versions): Improve reliability by removing remnances of Tumblr Live (#2988)
Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>
This commit is contained in:
parent
c72b1711be
commit
897b4dbce9
@ -1062,10 +1062,6 @@ public final class app/revanced/patches/tumblr/fixes/FixOldVersionsPatch : app/r
|
|||||||
public synthetic fun execute (Lapp/revanced/patcher/data/Context;)V
|
public synthetic fun execute (Lapp/revanced/patcher/data/Context;)V
|
||||||
}
|
}
|
||||||
|
|
||||||
public final class app/revanced/patches/tumblr/fixes/fingerprints/HttpPathParserFingerprint : app/revanced/patcher/fingerprint/MethodFingerprint {
|
|
||||||
public static final field INSTANCE Lapp/revanced/patches/tumblr/fixes/fingerprints/HttpPathParserFingerprint;
|
|
||||||
}
|
|
||||||
|
|
||||||
public final class app/revanced/patches/tumblr/live/DisableTumblrLivePatch : app/revanced/patcher/patch/BytecodePatch {
|
public final class app/revanced/patches/tumblr/live/DisableTumblrLivePatch : app/revanced/patcher/patch/BytecodePatch {
|
||||||
public static final field INSTANCE Lapp/revanced/patches/tumblr/live/DisableTumblrLivePatch;
|
public static final field INSTANCE Lapp/revanced/patches/tumblr/live/DisableTumblrLivePatch;
|
||||||
public fun execute (Lapp/revanced/patcher/data/BytecodeContext;)V
|
public fun execute (Lapp/revanced/patcher/data/BytecodeContext;)V
|
||||||
|
@ -5,34 +5,63 @@ import app.revanced.patcher.extensions.InstructionExtensions.addInstructions
|
|||||||
import app.revanced.patcher.patch.BytecodePatch
|
import app.revanced.patcher.patch.BytecodePatch
|
||||||
import app.revanced.patcher.patch.annotation.CompatiblePackage
|
import app.revanced.patcher.patch.annotation.CompatiblePackage
|
||||||
import app.revanced.patcher.patch.annotation.Patch
|
import app.revanced.patcher.patch.annotation.Patch
|
||||||
|
import app.revanced.patches.tumblr.fixes.fingerprints.AddQueryParamFingerprint
|
||||||
import app.revanced.patches.tumblr.fixes.fingerprints.HttpPathParserFingerprint
|
import app.revanced.patches.tumblr.fixes.fingerprints.HttpPathParserFingerprint
|
||||||
import app.revanced.util.exception
|
import app.revanced.util.exception
|
||||||
|
|
||||||
@Patch(
|
@Patch(
|
||||||
name = "Fix old versions",
|
name = "Fix old versions",
|
||||||
description = "Fixes old versions of the app (v33.2 and earlier) breaking due to Tumblr removing remnants of Tumblr" +
|
description = "Fixes old versions of the app (v33.2 and earlier) breaking due to Tumblr removing remnants of Tumblr" +
|
||||||
" Live from the API, which causes many requests to fail. This patch has no effect on newer versions of the app.",
|
" Live from the API, which causes many requests to fail. This patch has no effect on newer versions of the app.",
|
||||||
compatiblePackages = [CompatiblePackage("com.tumblr")],
|
compatiblePackages = [CompatiblePackage("com.tumblr")],
|
||||||
use = false,
|
use = false,
|
||||||
)
|
)
|
||||||
@Suppress("unused")
|
@Suppress("unused")
|
||||||
object FixOldVersionsPatch : BytecodePatch(
|
object FixOldVersionsPatch : BytecodePatch(
|
||||||
setOf(HttpPathParserFingerprint),
|
setOf(HttpPathParserFingerprint, AddQueryParamFingerprint),
|
||||||
) {
|
) {
|
||||||
override fun execute(context: BytecodeContext) =
|
override fun execute(context: BytecodeContext) {
|
||||||
|
val liveQueryParameters = listOf(
|
||||||
|
",?live_now",
|
||||||
|
",?live_streaming_user_id",
|
||||||
|
)
|
||||||
|
|
||||||
HttpPathParserFingerprint.result?.let {
|
HttpPathParserFingerprint.result?.let {
|
||||||
val endIndex = it.scanResult.patternScanResult!!.endIndex
|
val endIndex = it.scanResult.patternScanResult!!.endIndex
|
||||||
|
// Remove the live query parameters from the path when it's specified via a @METHOD annotation.
|
||||||
it.mutableMethod.addInstructions(
|
for (liveQueryParameter in liveQueryParameters) {
|
||||||
endIndex + 1,
|
it.mutableMethod.addInstructions(
|
||||||
"""
|
endIndex + 1,
|
||||||
# Remove "?live_now" from the request path p2.
|
"""
|
||||||
# p2 = p2.replace(p1, p3)
|
# urlPath = urlPath.replace(liveQueryParameter, "")
|
||||||
const-string p1, ",?live_now"
|
const-string p1, "$liveQueryParameter"
|
||||||
const-string p3, ""
|
const-string p3, ""
|
||||||
invoke-virtual {p2, p1, p3}, Ljava/lang/String;->replace(Ljava/lang/CharSequence;Ljava/lang/CharSequence;)Ljava/lang/String;
|
invoke-virtual {p2, p1, p3}, Ljava/lang/String;->replace(Ljava/lang/CharSequence;Ljava/lang/CharSequence;)Ljava/lang/String;
|
||||||
move-result-object p2
|
move-result-object p2
|
||||||
""",
|
""",
|
||||||
)
|
)
|
||||||
|
}
|
||||||
} ?: throw HttpPathParserFingerprint.exception
|
} ?: throw HttpPathParserFingerprint.exception
|
||||||
|
|
||||||
|
AddQueryParamFingerprint.result?.let {
|
||||||
|
// Remove the live query parameters when passed via a parameter which has the @Query annotation.
|
||||||
|
// e.g. an API call could be defined like this:
|
||||||
|
// @GET("api/me/info")
|
||||||
|
// ApiResponse getCurrentUserInfo(@Query("fields[blog]") String value)
|
||||||
|
// which would result in the path "api/me/inf0?fields[blog]=${value}"
|
||||||
|
// Here we make sure that this value doesn't contain the broken query parameters.
|
||||||
|
for (liveQueryParameter in liveQueryParameters) {
|
||||||
|
it.mutableMethod.addInstructions(
|
||||||
|
0,
|
||||||
|
"""
|
||||||
|
# queryParameterValue = queryParameterValue.replace(liveQueryParameter, "")
|
||||||
|
const-string v0, "$liveQueryParameter"
|
||||||
|
const-string v1, ""
|
||||||
|
invoke-virtual {p2, v0, v1}, Ljava/lang/String;->replace(Ljava/lang/CharSequence;Ljava/lang/CharSequence;)Ljava/lang/String;
|
||||||
|
move-result-object p2
|
||||||
|
""",
|
||||||
|
)
|
||||||
|
}
|
||||||
|
} ?: throw AddQueryParamFingerprint.exception
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@ -0,0 +1,11 @@
|
|||||||
|
package app.revanced.patches.tumblr.fixes.fingerprints
|
||||||
|
|
||||||
|
import app.revanced.patcher.fingerprint.MethodFingerprint
|
||||||
|
|
||||||
|
// Fingerprint for the addQueryParam method from retrofit2
|
||||||
|
// https://github.com/square/retrofit/blob/trunk/retrofit/src/main/java/retrofit2/RequestBuilder.java#L186
|
||||||
|
// Injecting here allows modifying dynamically set query parameters
|
||||||
|
internal object AddQueryParamFingerprint : MethodFingerprint(
|
||||||
|
strings = listOf("Malformed URL. Base: ", ", Relative: "),
|
||||||
|
parameters = listOf("Ljava/lang/String;", "Ljava/lang/String;", "Z"),
|
||||||
|
)
|
@ -3,13 +3,13 @@ package app.revanced.patches.tumblr.fixes.fingerprints
|
|||||||
import app.revanced.patcher.fingerprint.MethodFingerprint
|
import app.revanced.patcher.fingerprint.MethodFingerprint
|
||||||
import com.android.tools.smali.dexlib2.Opcode
|
import com.android.tools.smali.dexlib2.Opcode
|
||||||
|
|
||||||
// Fingerprint for the parseHttpMethodAndPath from retrofit2
|
// Fingerprint for the parseHttpMethodAndPath method from retrofit2
|
||||||
// https://github.com/square/retrofit/blob/ebf87b10997e2136af4d335276fa950221852c64/retrofit/src/main/java/retrofit2/RequestFactory.java#L270-L302
|
// https://github.com/square/retrofit/blob/ebf87b10997e2136af4d335276fa950221852c64/retrofit/src/main/java/retrofit2/RequestFactory.java#L270-L302
|
||||||
// Injecting here allows modifying the path/query params of API endpoints defined via annotations
|
// Injecting here allows modifying the path/query params of API endpoints defined via annotations
|
||||||
object HttpPathParserFingerprint : MethodFingerprint(
|
internal object HttpPathParserFingerprint : MethodFingerprint(
|
||||||
strings = listOf("Only one HTTP method is allowed. Found: %s and %s."),
|
strings = listOf("Only one HTTP method is allowed. Found: %s and %s."),
|
||||||
opcodes = listOf(
|
opcodes = listOf(
|
||||||
Opcode.IPUT_OBJECT,
|
Opcode.IPUT_OBJECT,
|
||||||
Opcode.IPUT_BOOLEAN
|
Opcode.IPUT_BOOLEAN,
|
||||||
)
|
),
|
||||||
)
|
)
|
||||||
|
Loading…
Reference in New Issue
Block a user