Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Browsing folders is broken in 28.0.5 if restrict usage is active #3670

Closed
MaxWaldorf opened this issue May 7, 2024 · 3 comments · Fixed by #3672
Closed

Browsing folders is broken in 28.0.5 if restrict usage is active #3670

MaxWaldorf opened this issue May 7, 2024 · 3 comments · Fixed by #3672

Comments

@MaxWaldorf
Copy link

Describe the bug
With version 8.3.6 on 28.0.5, if you activate this setting, you end up having users unable to browse through files and folders as interface will try to download the htm page each and every time...

To Reproduce
Steps to reproduce the behavior:

  1. Activate restrict usage to specific group and select one...
  2. Use a user that is outside the selected group
  3. Try to open a folder
  4. See issue

Expected behavior
User should be able to browse in the clicked folder...

Client details:

  • OS: Windows
  • Browser chrome like
  • Version (lastest)
  • Device: desktop

Server details

Operating system:
Docker image (community one)

Database:
Postgresql 13

Nextcloud version:
28.0.5

Version of the richdocuments app
8.3.6

@pierre-alain-b
Copy link

pierre-alain-b commented May 8, 2024

Hi there, I have exactly the same issue with a manually installed Nextcloud instance. For users outside of the groups authorized to use the app, the user interface is broken: no folder or file can be opened - the .html file that is downloaded contains these sentences in raw text: "This is the WebDAV interface. It can only be accessed by WebDAV clients such as the Nextcloud desktop sync client.".

Here is the error message that pops up (multiple times) in the browser console:

TypeError: dt is undefined
    enabled file-actions.js:26
    enabledActions FileEntryActions.vue:70
    enabledActions FileEntryActions.vue:70
    get core-common.js:2
    evaluate core-common.js:2
    Br core-common.js:2
    enabledDefaultActions FileEntryActions.vue:89
    get core-common.js:2
    evaluate core-common.js:2
    Br core-common.js:2
    linkTo FileEntryName.vue:80
    get core-common.js:2
    evaluate core-common.js:2
    Br core-common.js:2
    Gr FileEntryName.vue:1
    _render core-common.js:2
    r core-common.js:2
    get core-common.js:2
    run core-common.js:2
    nr core-common.js:2
    yn core-common.js:2
    gn core-common.js:2
    promise callback*hn core-common.js:2
    yn core-common.js:2
    rr core-common.js:2
    update core-common.js:2
    notify core-common.js:2
    set core-common.js:2
    set value core-common.js:2
    set core-common.js:2
    Oe core-common.js:2
    updateNodes files.ts:39
    _ pinia.mjs:1380
    fetchContent FilesList.vue:321
    _unsubscribeStore FilesList.vue:293
    o pinia.mjs:1420
    un core-common.js:2
    p core-common.js:2
    run core-common.js:2
    nr core-common.js:2
    yn core-common.js:2
    gn core-common.js:2
    promise callback*hn core-common.js:2
    yn core-common.js:2
    rr core-common.js:2
    update core-common.js:2
    notify core-common.js:2
    set core-common.js:2
    set core-common.js:2
    resizeObserver VirtualList.vue:146
    l core-common.js:2
    setTimeout handler*u core-common.js:2
    mounted VirtualList.vue:144
    un core-common.js:2
    Zn core-common.js:2
    insert core-common.js:2
    k core-common.js:2
    ki core-common.js:2
    _update core-common.js:2
    r core-common.js:2
    get core-common.js:2
    run core-common.js:2
    nr core-common.js:2
    yn core-common.js:2
    gn core-common.js:2
    promise callback*hn core-common.js:2
    yn core-common.js:2
    rr core-common.js:2
    update core-common.js:2
    notify core-common.js:2
    Oe core-common.js:2
    setRoot files.ts:49
    _ pinia.mjs:1380
    fetchContent FilesList.vue:327
    mounted FilesList.vue:288
    un core-common.js:2
    Zn core-common.js:2
    insert core-common.js:2
    k core-common.js:2
    ki core-common.js:2
    _update core-common.js:2
    r core-common.js:2
    get core-common.js:2
    e core-common.js:2
    mount core-common.js:2
    $mount core-common.js:2
    16669 main.ts:33
    Webpack 5
core-common.js:2:2116787

image

@juliushaertl
Copy link
Member

Thanks for reporting. I pushed a fix for this in #3672, testing would be very welcome.

@pierre-alain-b
Copy link

I've tested the new file in #3672 and I confirm that it works: namely nextcloud office works for authorized users and files/folders are fully accessible for users not authorized to use the app. Thank you for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants