8000 Fix permission issue with chunk upload by lohanidamodar · Pull Request #7328 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix permission issue with chunk upload #7328

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Jan 2, 2024
Merged

Fix permission issue with chunk upload #7328

merged 6 commits into from
Jan 2, 2024

Conversation

lohanidamodar
Copy link
Member
@lohanidamodar lohanidamodar commented Dec 25, 2023 •
edited
Loading

What does this PR do?

  • Validates create permission while updating chunks in chunk upload

Test Plan

  • Update existing large file upload test to not include update permission

Related PRs and Issues

  • (Related PR or issue)

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@stnguyen90 stnguyen90 self-requested a review December 28, 2023 00:35
@github-actions github-actions bot mentioned this pull request Jan 1, 2024
Closed
@lohanidamodar lohanidamodar merged commit 3236e9e into main Jan 2, 2024
@lohanidamodar lohanidamodar deleted the fix-chunk-upload-permission branch January 2, 2024 06:51
@lohanidamodar lohanidamodar restored the fix-chunk-upload-permission branch January 2, 2024 08:21
@stnguyen90 stnguyen90 deleted the fix-chunk-upload-permission branch January 2, 2024 15:21
@github-actions github-actions bot mentioned this pull request Feb 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eldadfux eldadfux eldadfux approved these changes

@stnguyen90 stnguyen90 Awaiting requested review from stnguyen90

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lohanidamodar @eldadfux
0