Bug 133011 - NextCloud integration: document opening failed after creation
Summary: NextCloud integration: document opening failed after creation
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice Online
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-13 18:53 UTC by Marco Marinello
Modified: 2020-08-07 15:09 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marco Marinello 2020-05-13 18:53:12 UTC
As reported here https://github.com/nextcloud/richdocuments/issues/956#event-3332153338 , from chromium, when creating a new document, the page of LOOL loads but shows "unsuccessful load of the document" message. The file is being created. Back to the folder and click on the file works fine.

Browser log

bundle.js:54 Uncaught DOMException: Blocked a frame with origin "https://cloudpub-lool.continuity.space" from accessing a cross-origin frame.
    at NewClass._postMessageListener (https://lool.domain.tld/loleaflet/87fea7126/bundle.js:54:18860)
    at handler (https://lool.domain.tld/loleaflet/87fea7126/bundle.js:52:22366)

The LOOL image is built upon online@libreoffice-6-4 and core@libreoffice6-4-4
Comment 1 Aron Budea 2020-05-15 03:24:13 UTC
Thanks for reporting the issue here, the question still is, have you tried/can you try with a LOOL master build? I'd guess this is fixed by the following commit:
https://cgit.freedesktop.org/libreoffice/online/commit/?id=b5f58fbe8c4f283d10def377c8a687245312fa1f
Comment 2 Marco Marinello 2020-05-15 07:42:52 UTC
I could but I think I just don't want. We build from branches because they're supposed to be stable, not the same for master. Is it possible to backport that commit onto libreoffice-6-4?
Comment 3 Aron Budea 2020-05-17 14:43:58 UTC
Sure, go ahead. For the record, I've only made a semi-educated guess about the fix, and have no means to reproduce this bug.
Comment 4 Marco Marinello 2020-05-21 14:04:24 UTC
Tried to backport the mentioned commit, the result is that lool won't even boot: I mean, the process works fine but when trying to open the document it simply stays with the grey band and nothing else.
Comment 5 Marco Marinello 2020-08-07 08:43:38 UTC
The same bug is now reproducible even with Firefox on LOOL 7.0 with the version 3.7.3 of the integration.

Steps to reproduce:
- login to NextCloud
- create a LOOL document
should fail (loading failed)

go back to nextcloud and open the file: works
Comment 6 Aron Budea 2020-08-07 13:26:47 UTC
Do you see the same error in the browser console?
Comment 7 Marco Marinello 2020-08-07 13:41:34 UTC
No, here is the full output in Chromium console after clicking the "create" button:

globals.js:66 The humanFileSize library is deprecated! It will be removed in nextcloud 19.
ne @ globals.js:66
r @ globals.js:82
_createRow @ merged-index.js?v=611022ef-49:5484
fileList._createRow @ merged-index.js?v=611022ef-49:12136
e._createRow @ filesplugin.js:70
e._createRow @ share.js:57
_renderRow @ merged-index.js?v=611022ef-49:5668
add @ merged-index.js?v=611022ef-49:5609
(anonymous) @ files.js:304
u @ jquery.js:3099
fireWith @ jquery.js:3211
x @ jquery.js:8264
(anonymous) @ jquery.js:8605
load (async)
send @ jquery.js:8622
ajax @ jquery.js:8166
m.each.m.<computed> @ jquery.js:8311
_createDocumentFromTemplate @ files.js:299
(anonymous) @ files.js:326
(anonymous) @ jquery.js:3256
u @ jquery.js:3099
fireWith @ jquery.js:3211
x @ jquery.js:8264
(anonymous) @ jquery.js:8605
load (async)
send @ jquery.js:8622
ajax @ jquery.js:8166
_openTemplatePicker @ files.js:320
actionHandler @ files.js:241
(anonymous) @ merged-index.js?v=611022ef-49:11116
dispatch @ jquery.js:4435
v.handle @ jquery.js:4121
trigger @ jquery.js:4350
e.event.trigger @ jquery-migrate.min.js:2
(anonymous) @ jquery.js:4901
each @ jquery.js:374
each @ jquery.js:139
trigger @ jquery.js:4900
m.each.m.fn.<computed> @ jquery.js:7487
(anonymous) @ merged-index.js?v=611022ef-49:11093
dispatch @ jquery.js:4435
v.handle @ jquery.js:4121
jquery-migrate.min.js:2 JQMIGRATE: Migrate is installed, version 1.4.1
ProxyBus.js:65 Proxying an event bus of version 1.1.2 with 0.2.1
t @ ProxyBus.js:65
(anonymous) @ index.js:20
n @ bootstrap:19
(anonymous) @ requesttoken.js:11
n @ bootstrap:19
(anonymous) @ index.js:25
n @ bootstrap:19
(anonymous) @ client.js:12
n @ bootstrap:19
(anonymous) @ notifications.js?v=611022ef-49:21
n @ bootstrap:19
(anonymous) @ bootstrap:83
(anonymous) @ notifications.js?v=611022ef-49:1
globals.js:66 Handlebars is deprecated: please ship your own, this will be removed in Nextcloud 20
ne @ globals.js:66
get @ globals.js:96
e.default @ no-conflict.js:8
(anonymous) @ handlebars.runtime.js:60
n @ bootstrap:19
(anonymous) @ runtime.js:3
n @ bootstrap:19
(anonymous) @ file.handlebars:1
n @ bootstrap:19
(anonymous) @ richObjectStringParser.js:13
n @ bootstrap:19
(anonymous) @ bootstrap:83
(anonymous) @ notifications.js?v=611022ef-49:1
App.vue:253 Notifications permissions not yet requested
loleaflet.html?WOPISrc=https%3A%2F%2Fmy.cloud.eu%2Findex.php%2Fapps%2Frichdocuments%2Fwopi%2Ffiles%2F97456_oca9pkmjij5d&title=test2.odt&lang=it&closebutton=1&revisionhistory=1:70 GET https://lool.cloud.eu/loleaflet/30aa94789/branding.css net::ERR_ABORTED 404
loleaflet.html?WOPISrc=https%3A%2F%2Fmy.cloud.eu%2Findex.php%2Fapps%2Frichdocuments%2Fwopi%2Ffiles%2F97456_oca9pkmjij5d&title=test2.odt&lang=it&closebutton=1&revisionhistory=1:221 GET https://lool.cloud.eu/loleaflet/30aa94789/branding.js net::ERR_ABORTED 404
my.cloud.eu/:1 GET https://my.cloud.eu/index.php/core/preview.png?file=%2F%2Ftest2.odt&c=59f8331f043cf1c65ad9be772bedd32b&x=250&y=250&forceIcon=0 404 (Not Found)
loleaflet.html?WOPISrc=https%3A%2F%2Fmy.cloud.eu%2Findex.php%2Fapps%2Frichdocuments%2Fwopi%2Ffiles%2F97456_oca9pkmjij5d&title=test2.odt&lang=it&closebutton=1&revisionhistory=1:216 GET https://lool.cloud.eu/loleaflet/30aa94789/branding-desktop.css net::ERR_ABORTED 404
(anonymous) @ loleaflet.html?WOPISrc=https%3A%2F%2Fmy.cloud.eu%2Findex.php%2Fapps%2Frichdocuments%2Fwopi%2Ffiles%2F97456_oca9pkmjij5d&title=test2.odt&lang=it&closebutton=1&revisionhistory=1:216
bundle.js:21 [Deprecation] Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
request_JSON @ bundle.js:21
String_ctr.<computed> @ bundle.js:21
(anonymous) @ bundle.js:21
(anonymous) @ bundle.js:21
bundle.js:39 Blocked autofocusing on a form control in a cross-origin subframe.
Comment 8 Aron Budea 2020-08-07 14:17:26 UTC
Excellent, I'm assuming the original issue is solved, and what you have now is different, please open a new bug report.