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

Multiple fixes found when running web with CERNBox #11946

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

diocas
Copy link
Contributor

@diocas diocas commented Nov 21, 2024

The fixes are described in the changelog.
Most of the cases are error handling of things that your server returns but ours might not. But, for example, the lack of parent ID should be treated by Web, since in the code is marked as optional, so I would argue it is not correct to just crash if not set. Same for TUS support, if not configured it just crashed to upload, which should not.

For the preview service, I think that when the object was created, the store was not loaded yet. So, whatever config I returned in the capabilities, the listing of files would always render all the hardcoded mime types.
But let me know if you have a proper way/different way of solving the issue.

More to come!

The results were not being taken into account.
When TUS is not supported by the backend, web crashed trying to see if the user had permissions to upload. Even if they did, it was being denied due to that crash.
…not report their parent

The parameter Resource.parentFolderId is optional, so it should be treated as so in other places.
The configuration was being ignored.
The current implementation hardcodes this and assumes that the personal volume always starts with `personal/`, which might not be the case.
@diocas diocas requested a review from kulmann November 21, 2024 10:07
@diocas diocas force-pushed the up_fixes1 branch 2 times, most recently from cd7c9da to 10f9e93 Compare November 21, 2024 10:17
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 this pull request may close these issues.

1 participant