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

Responses are rendered in different places #5085

Closed
labkode opened this issue May 7, 2021 · 15 comments · Fixed by #5917
Closed

Responses are rendered in different places #5085

labkode opened this issue May 7, 2021 · 15 comments · Fixed by #5917
Labels
Category:Defect Existing functionality is not working as expected Early-Adopter:CERN Type:Bug Something isn't working

Comments

@labkode
Copy link

labkode commented May 7, 2021

If I send a request to list a large directory /A/B/C.
In the mean time I change path to /Z.
Request to /A/B/C finished and shows the content of C inside Z.

The expected behaviour is that the response should not be rendered as this is not the original path it was requested.

@kulmann kulmann added the Category:Defect Existing functionality is not working as expected label May 12, 2021
@pascalwengerter
Copy link
Contributor

@labkode could you check if this is still the case? Should have been fixed by our recent changes

@pascalwengerter
Copy link
Contributor

Closed as fixed (due to various performance improvements) unless @labkode can reproduce

@diocas
Copy link
Contributor

diocas commented Oct 4, 2021

Hi, we managed to reproduce this error again. As described in #5875, loading the / root dir (which takes some time) before loading the homedir, is causing the homedir contents to be replaced. And then this is extremely confusing to users because clicking "All my files" does nothing in this case.

@diocas diocas reopened this Oct 4, 2021
@labkode
Copy link
Author

labkode commented Oct 8, 2021

As discussed today this is major issue in general. To short-cut it we need to disable the PROPFIND to root when loading the UI. This is a major issue for the web and I hope it can be fixed without having to change framework.

@labkode
Copy link
Author

labkode commented Oct 14, 2021

@kulmann @dragotin
This is a major blocker for our roll-out and needs to be fixed in short-time (days).
By manually testing it you end up with: browsing into a directory and is empty, browsing into a directory that contains contents from other places, clicking on shared with me and listing your home files rather than your shares, etc ...

@kulmann
Copy link
Member

kulmann commented Oct 15, 2021

@kulmann @dragotin
This is a major blocker for our roll-out and needs to be fixed in short-time (days).
By manually testing it you end up with: browsing into a directory and is empty, browsing into a directory that contains contents from other places, clicking on shared with me and listing your home files rather than your shares, etc ...

Thank you for your comment, I was finally able to reproduce it locally. We only solved edge cases within one view so far (so if you navigate into sub- or parent-folders in the All files list). That all works fine now as far as I know, because those requests are resolved sequentially. BUT: across views requests for loading resources can indeed overlap. So a slow loading "shared with me" will render into a fast loading "All files" view and vice versa. "render" is not 100% correct because there are also some error cases, where you'll see "resource not found". With that I'll shift priority back onto #5892 immediately (now).

@fschade
Copy link
Collaborator

fschade commented Oct 17, 2021

@labkode can you please test #5917. Should fix it, i hope i spotted all places.

@diocas
Copy link
Contributor

diocas commented Oct 18, 2021

I've tested it and it looks better. But I still manage to reproduce the issue.

One of the times I reproduced, I don't remember what I did. But I got a "resource not found", like when in the past we switched views (but I might have clicked in an empty folder as well... Don't know).

In another, in the files/list/all, I clicked the breadcrumb to go to a higher level (with takes some time to load), then hit the "back" button of my browser. The route changed, the request started and finished, but the page kept loading, waiting for the upper level request. When that one finished, the page rendered with its contents, but the route+breadcrumb was not matching the content.

This leads me to conclude that although it seems that seeing this issue when switching between views has become hard (or impossible), it's still easy to reproduce when in the same view.

@fschade
Copy link
Collaborator

fschade commented Oct 19, 2021

@diocas, good catch, thank you. It was implemented for switchings views, but not for switching a resource in the same view.
I fixed it, can you please have a second look.

@diocas
Copy link
Contributor

diocas commented Oct 20, 2021

@fschade tested and looks ok!

@fschade
Copy link
Collaborator

fschade commented Oct 20, 2021

thanks @diocas

@fschade fschade closed this as completed Oct 20, 2021
@elizavetaRa elizavetaRa reopened this Aug 30, 2022
@elizavetaRa
Copy link
Member

We can reproduce this issue again
Files shared with me - CERNBox

@diocas diocas added the Type:Bug Something isn't working label Aug 30, 2022
@diocas
Copy link
Contributor

diocas commented Aug 30, 2022

ping @kulmann

@kulmann
Copy link
Member

kulmann commented Aug 31, 2022

Please provide steps to reproduce. If I do what you show in the screen capture, even with throttled connection and large folders, I can't reproduce it.

@dragotin
Copy link
Contributor

Seems fixed, if not @elizavetaRa please reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Defect Existing functionality is not working as expected Early-Adopter:CERN Type:Bug Something isn't working
Projects
None yet
7 participants