Project space have public links from the personal space

[ScharfViktor]

ownCloud Web UI 5.7.0-rc.5:

Prerequisite:

• create a folder with the file in your personal space
• create a public link to the folder

Steps:

• create new project space
• go to the members

Actual: here we have a public link from personal space

Screen.Recording.2022-07-12.at.14.28.13.mov

[pascalwengerter]

Can confirm, this is critical and needs investigation

[pascalwengerter]

Can confirm, this is critical and needs investigation

When investigating, I have started the recently released beta.4

• with the embedded web it brings and the problem does not arise
• with the current web master (aka close to v5.7.0-rc.5) and the problem shows up
  which leads me to the conclusion that this is a frontend issue, most likely related to either the auth (or resharing?) refactorings

[pascalwengerter]

Can confirm, this is critical and needs investigation

When investigating, I have started the recently released beta.4

* with the embedded `web` it brings and the problem does not arise

* with the current `web` master (aka close to `v5.7.0-rc.5`) and the problem shows up
  which leads me to the conclusion that this is a frontend issue, most likely related to either the auth (or resharing?) refactorings

Checked out 434a622 (merge commit of the auth PR) and the issue is present there (resharing was merged a day later IIRC so it shouldn't be the culprint)

[pascalwengerter]

Can confirm, this is critical and needs investigation

When investigating, I have started the recently released beta.4

* with the embedded `web` it brings and the problem does not arise

* with the current `web` master (aka close to `v5.7.0-rc.5`) and the problem shows up
  which leads me to the conclusion that this is a frontend issue, most likely related to either the auth (or resharing?) refactorings

Checked out 434a622 (merge commit of the auth PR) and the issue is present there (resharing was merged a day later IIRC so it shouldn't be the culprint)

Also checked 94b0538 which was right before the auth PR got merged and it's fine there, so I guess #7072 caused the regression

[pascalwengerter]

Can confirm, this is critical and needs investigation

When investigating, I have started the recently released beta.4

* with the embedded `web` it brings and the problem does not arise

* with the current `web` master (aka close to `v5.7.0-rc.5`) and the problem shows up
  which leads me to the conclusion that this is a frontend issue, most likely related to either the auth (or resharing?) refactorings

Checked out 434a622 (merge commit of the auth PR) and the issue is present there (resharing was merged a day later IIRC so it shouldn't be the culprint)

Also checked 94b0538 which was right before the auth PR got merged and it's fine there, so I guess #7072 caused the regression

Looks like currentFolder.path is an empty string in

web/packages/web-app-files/src/services/folder/spaces/loaderProject.ts

Line 74 in 434a622

path: currentFolder.path

which leads to all shares being fetched upon navigating into a project space?

[kulmann]

web/packages/web-app-files/src/components/SideBar/Shares/SharesPanel.vue

Line 63 in 1ad492a

storageId: this.highlightedFile.fileId,

tries to use the fileId property of a resource for loading shares, but that field is empty for spaces, see

web/packages/web-app-files/src/helpers/resources.ts

Line 149 in 1ad492a

fileId: '',