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

Fixes for getting the filename in the FileInfo class #43652

Merged

Conversation

hopleus
Copy link
Contributor

@hopleus hopleus commented Feb 19, 2024

Summary

Fixes for getting the filename in the case when the file name key is received, but it is empty.

P.S. Faced with such a problem with ObjectStore enabled in S3 mode

Checklist

@hopleus hopleus force-pushed the bugfix/fixed-get-filename-in-fileinfo branch from 855e56e to b02abe6 Compare February 19, 2024 12:14
@hopleus
Copy link
Contributor Author

hopleus commented Feb 19, 2024

/backport to stable28

@hopleus
Copy link
Contributor Author

hopleus commented Feb 19, 2024

/backport to stable27

@hopleus
Copy link
Contributor Author

hopleus commented Feb 19, 2024

/backport to stable26

@nextcloud nextcloud deleted a comment from hopleus Feb 19, 2024
@solracsf solracsf added the 3. to review Waiting for reviews label Feb 19, 2024
@solracsf solracsf added this to the Nextcloud 29 milestone Feb 19, 2024
@solracsf solracsf requested review from a team, icewind1991, nfebe and sorbaugh and removed request for a team February 19, 2024 12:58
@hopleus hopleus force-pushed the bugfix/fixed-get-filename-in-fileinfo branch from b02abe6 to 1c085a9 Compare February 19, 2024 13:25
@hopleus
Copy link
Contributor Author

hopleus commented Feb 29, 2024

@solracsf Hi. Could you run the tests again?

P.S. Last time cypress crashed with 503 errors and I didn't understand why

@Altahrim Altahrim mentioned this pull request Mar 12, 2024
@hopleus hopleus requested a review from Altahrim March 12, 2024 14:26
@Altahrim Altahrim mentioned this pull request Mar 14, 2024
@skjnldsv skjnldsv merged commit f316edb into nextcloud:master Mar 14, 2024
148 of 159 checks passed
Copy link

welcome bot commented Mar 14, 2024

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

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

Successfully merging this pull request may close these issues.

5 participants