[Bug]: Plugin call after navigation can be answered before reply proxy is updated

Offen Anfängerfreundlich
#8,616 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
84/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
android, java
Bereich
mobile-dev

Rechercherichtung

Beginne in MessageHandler.java und untersuche die Reihenfolge der Aktualisierung des Reply-Proxys und des Postens der Nachricht während der Navigation oder des Reloads. Reproduziere das Problem mit einem schnellen Preferences-Lesevorgang auf Android und verifiziere anschließend, dass Plugin-Aufrufe von der neuen Seite den aktualisierten Proxy verwenden und aufgelöst werden, anstatt stillschweigend verworfen zu werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

triage
Capacitor Version

@capacitor/cli: 8.4.0
@capacitor/core: 8.4.0
@capacitor/android: 8.4.0

Other API Details
npm: 11.0.0
node: 22.13.0
Platforms Affected
  • iOS
  • Android
  • Web
Current Behavior

After a reload or navigation, a fast plugin call (e.g. a Preferences read, which is how the bug appeared for us) can potentially never resolve as it uses the previous proxy instead of the updated one. The cause are these lines in MessageHandler.java:

if (isMainFrame) {
    postMessage(message.getData());
    javaScriptReplyProxy = replyProxy;
}

In case of a fast plugin call, sendResponseMessage() is reached before the proxy is updated, and therefore still uses the previous proxy and is silently dropped. Later calls work, and there's also no issue upon first page load as, at this point, javaScriptReplyProxy is null and the old evaluateJavascript path is used instead.

Expected Behavior

All plugin calls of the new page should go through the updated proxy.

Project Reproduction

Reproduced in a private repo

Additional Information

The bug is avoidable with android.useLegacyBridge = true as this routes every response through evaluateJavascriptand avoids the proxy entirely.

I've locally fixed the bug by swapping the two lines above, i.e., making sure the proxy is updated before the message post.

Vorherrschende Sprache
TypeScript
Sterne
16.7k
Forks
1.3k
Ø Merge
3 T. 10 Std.
Gemergte PRs (30 T.)
10

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus ionic-team/capacitor

Alle Issues in ionic-team/capacitor

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.