mirror of
https://github.com/revanced/revanced-integrations.git
synced 2025-02-02 15:17:32 +01:00
fix(youtube/client-spoof): update settings text for known side effects of spoof signature (#424)
This commit is contained in:
LisoUseInAIKyrios
GitHub
parent
db9b0e312d
commit
d7a3973ef1
@ -1,14 +1,12 @@
|
|||||||
package app.revanced.integrations.patches;
|
package app.revanced.integrations.patches;
|
||||||
|
|
||||||
import androidx.annotation.NonNull;
|
import static app.revanced.integrations.utils.ReVancedUtils.containsAny;
|
||||||
import androidx.annotation.Nullable;
|
|
||||||
import app.revanced.integrations.settings.SettingsEnum;
|
import app.revanced.integrations.settings.SettingsEnum;
|
||||||
import app.revanced.integrations.shared.PlayerType;
|
import app.revanced.integrations.shared.PlayerType;
|
||||||
import app.revanced.integrations.utils.LogHelper;
|
import app.revanced.integrations.utils.LogHelper;
|
||||||
import app.revanced.integrations.utils.ReVancedUtils;
|
import app.revanced.integrations.utils.ReVancedUtils;
|
||||||
|
|
||||||
import static app.revanced.integrations.utils.ReVancedUtils.containsAny;
|
|
||||||
|
|
||||||
public class SpoofSignatureVerificationPatch {
|
public class SpoofSignatureVerificationPatch {
|
||||||
/**
|
/**
|
||||||
* Enable/disable all workarounds that are required due to signature spoofing.
|
* Enable/disable all workarounds that are required due to signature spoofing.
|
||||||
@ -37,28 +35,6 @@ public class SpoofSignatureVerificationPatch {
|
|||||||
"SAFg" // Autoplay in scrim
|
"SAFg" // Autoplay in scrim
|
||||||
};
|
};
|
||||||
|
|
||||||
/**
|
|
||||||
* On app first start, the first video played usually contains a single non-default window setting value
|
|
||||||
* and all other subtitle settings for the video are (incorrect) default shorts window settings.
|
|
||||||
* For this situation, the shorts settings must be replaced.
|
|
||||||
*
|
|
||||||
* But some videos use multiple text positions on screen (such as https://youtu.be/3hW1rMNC89o),
|
|
||||||
* and by chance many of the subtitles uses window positions that match a default shorts position.
|
|
||||||
* To handle these videos, selectively allowing the shorts specific window settings to 'pass thru' unchanged,
|
|
||||||
* but only if the video contains multiple non-default subtitle window positions.
|
|
||||||
*
|
|
||||||
* Do not enable 'pass thru mode' until this many non default subtitle settings are observed for a single video.
|
|
||||||
*/
|
|
||||||
private static final int NUMBER_OF_NON_DEFAULT_SUBTITLES_BEFORE_ENABLING_PASSTHRU = 2;
|
|
||||||
|
|
||||||
/**
|
|
||||||
* The number of non default subtitle settings encountered for the current video.
|
|
||||||
*/
|
|
||||||
private static int numberOfNonDefaultSettingsObserved;
|
|
||||||
|
|
||||||
@Nullable
|
|
||||||
private static String currentVideoId;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Injection point.
|
* Injection point.
|
||||||
*
|
*
|
||||||
@ -121,115 +97,4 @@ public class SpoofSignatureVerificationPatch {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Last WindowsSetting constructor values. Values are checked for changes to reduce log spam.
|
|
||||||
*/
|
|
||||||
private static int lastAp, lastAh, lastAv;
|
|
||||||
private static boolean lastVs, lastSd;
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Injection point. Overrides values passed into SubtitleWindowSettings constructor.
|
|
||||||
*
|
|
||||||
* @param ap anchor position. A bitmask with 6 bit fields, that appears to indicate the layout position on screen
|
|
||||||
* @param ah anchor horizontal. A percentage [0, 100], that appears to be a horizontal text anchor point
|
|
||||||
* @param av anchor vertical. A percentage [0, 100], that appears to be a vertical text anchor point
|
|
||||||
* @param vs appears to indicate if subtitles exist, and the value is always true.
|
|
||||||
* @param sd function is not entirely clear
|
|
||||||
*/
|
|
||||||
public static int[] getSubtitleWindowSettingsOverride(int ap, int ah, int av, boolean vs, boolean sd) {
|
|
||||||
final boolean signatureSpoofing = SettingsEnum.SPOOF_SIGNATURE_VERIFICATION.getBoolean();
|
|
||||||
if (SettingsEnum.DEBUG.getBoolean()) {
|
|
||||||
if (ap != lastAp || ah != lastAh || av != lastAv || vs != lastVs || sd != lastSd) {
|
|
||||||
LogHelper.printDebug(() -> "video: " + VideoInformation.getVideoId() + " spoof: " + signatureSpoofing
|
|
||||||
+ " ap:" + ap + " ah:" + ah + " av:" + av + " vs:" + vs + " sd:" + sd);
|
|
||||||
lastAp = ap;
|
|
||||||
lastAh = ah;
|
|
||||||
lastAv = av;
|
|
||||||
lastVs = vs;
|
|
||||||
lastSd = sd;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!WORKAROUND) {
|
|
||||||
return new int[]{ap, ah, av};
|
|
||||||
}
|
|
||||||
|
|
||||||
// Videos with custom captions that specify screen positions appear to always have correct screen positions (even with spoofing).
|
|
||||||
// But for auto generated and most other captions, the spoof incorrectly gives various default Shorts caption settings.
|
|
||||||
// Check for these known default shorts captions parameters, and replace with the known correct values.
|
|
||||||
//
|
|
||||||
// If a regular video uses a custom subtitle setting that match a default short setting,
|
|
||||||
// then this will incorrectly replace the setting.
|
|
||||||
// But, if the video uses multiple subtitles in different screen locations, then detect the non-default values
|
|
||||||
// and do not replace any window settings for the video (regardless if they match a shorts default).
|
|
||||||
if (signatureSpoofing && !PlayerType.getCurrent().isNoneOrHidden()
|
|
||||||
&& numberOfNonDefaultSettingsObserved < NUMBER_OF_NON_DEFAULT_SUBTITLES_BEFORE_ENABLING_PASSTHRU) {
|
|
||||||
for (SubtitleWindowReplacementSettings setting : SubtitleWindowReplacementSettings.values()) {
|
|
||||||
if (setting.match(ap, ah, av, vs, sd)) {
|
|
||||||
return setting.replacementSetting();
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
numberOfNonDefaultSettingsObserved++;
|
|
||||||
LogHelper.printDebug(() ->
|
|
||||||
numberOfNonDefaultSettingsObserved < NUMBER_OF_NON_DEFAULT_SUBTITLES_BEFORE_ENABLING_PASSTHRU
|
|
||||||
? "Non default subtitle found."
|
|
||||||
: "Multiple non default subtitles found. Allowing all subtitles for this video to pass thru unchanged.");
|
|
||||||
}
|
|
||||||
|
|
||||||
return new int[]{ap, ah, av};
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Injection point.
|
|
||||||
*/
|
|
||||||
public static void setCurrentVideoId(@NonNull String videoId) {
|
|
||||||
try {
|
|
||||||
if (videoId.equals(currentVideoId)) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
currentVideoId = videoId;
|
|
||||||
numberOfNonDefaultSettingsObserved = 0;
|
|
||||||
} catch (Exception ex) {
|
|
||||||
LogHelper.printException(() -> "setCurrentVideoId failure", ex);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Known incorrect default Shorts subtitle parameters, and the corresponding correct (non-Shorts) values.
|
|
||||||
*/
|
|
||||||
private enum SubtitleWindowReplacementSettings {
|
|
||||||
DEFAULT_SHORTS_PARAMETERS_1(10, 50, 0, true, false,
|
|
||||||
34, 50, 95),
|
|
||||||
DEFAULT_SHORTS_PARAMETERS_2(9, 20, 0, true, false,
|
|
||||||
34, 50, 90),
|
|
||||||
DEFAULT_SHORTS_PARAMETERS_3(9, 20, 0, true, true,
|
|
||||||
33, 20, 100);
|
|
||||||
|
|
||||||
// original values
|
|
||||||
final int ap, ah, av;
|
|
||||||
final boolean vs, sd;
|
|
||||||
|
|
||||||
// replacement int values
|
|
||||||
final int[] replacement;
|
|
||||||
|
|
||||||
SubtitleWindowReplacementSettings(int ap, int ah, int av, boolean vs, boolean sd,
|
|
||||||
int replacementAp, int replacementAh, int replacementAv) {
|
|
||||||
this.ap = ap;
|
|
||||||
this.ah = ah;
|
|
||||||
this.av = av;
|
|
||||||
this.vs = vs;
|
|
||||||
this.sd = sd;
|
|
||||||
this.replacement = new int[]{replacementAp, replacementAh, replacementAv};
|
|
||||||
}
|
|
||||||
|
|
||||||
boolean match(int ap, int ah, int av, boolean vs, boolean sd) {
|
|
||||||
return this.ap == ap && this.ah == ah && this.av == av && this.vs == vs && this.sd == sd;
|
|
||||||
}
|
|
||||||
|
|
||||||
int[] replacementSetting() {
|
|
||||||
return replacement;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user