nextcloud/richdocuments

Crash when nextcloud is inside an iframe

Offen

#3.209 geöffnet am 30.09.2023

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (145 Forks)auto 404
1. to developbuggood first issue

Repository-Metriken

Stars
 (452 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Describe the bug When Nextcloud is inside an iframe, the new code for a dark interface crashes the richdocuments app.

To Reproduce Steps to reproduce the behavior:

  1. Have nextcloud in an iframe
  2. Open a richdocument file

Expected behavior The document should open

Client details:

  • OS: archlinux
  • Browser firefox
  • Version: 118.0
  • Device: desktop

Server details

Operating system: Debian 11

Web server: Apache

Database: Mysql

PHP version: 8.2

Nextcloud version: 27.1.1

Version of the richdocuments app 8.2

Version of Collabora Online 22.05.12.2

Browser log

The javascript console log :

Permission denied to access property "document" on cross-origin object

Contributor Guide

Research-Richtung
Untersuche die Verwendung von 'parent.document.body.dataset' in den Zeilen 35-36 von src/helpers/coolParameters.js. Innerhalb eines iframes kann der Zugriff auf parent.document aufgrund von Cross Origin Beschränkungen fehlschlagen. Erwäge zu prüfen, ob das übergeordnete Element dieselbe Herkunft hat, oder verwende eine andere Methode zur Erkennung des dunklen Modus. Suche nach Alternativen wie der Kommunikation mit dem übergeordneten Element per postMessage oder der Überprüfung eines vom übergeordneten Element gesetzten Cookies/localStorage.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptiframe security
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70