Large file upload response handling
@SilasKenneth is already working on this.
Since Jun 5, 2023.
Assessment
This issue has not been assessed yet.
Description
We need to handle different "completion" scenarios for Large File Uploads:
- OneDrive returns a response object which we currently don't deserialize correctly
- Outlook returns a 201 with no response body & no content-type header causing a deserialization error. Response contains a Location header with a URL to the attachment
- Deserialize error responses from the API
- Dominant language
- PHP
- Stars
- 16
- Forks
- 8
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoftgraph/msgraph-sdk-php-core
-
status:waiting-for-triage type:bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
microsoftgraph/msgraph-sdk-php-core#230 · 1 comment ·
-
status:waiting-for-triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
microsoftgraph/msgraph-sdk-php-core#227 · 2 comments · 1 reaction ·
-
status:waiting-for-triage type:bug
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
area:authentication goodfirstissue type:enhancement
microsoftgraph/msgraph-sdk-php-core#202 · 4 comments · 1 assignee ·
-
type:feature
Difficulty 4/5 3-5 days Newbie friendliness 35/100
microsoftgraph/msgraph-sdk-php-core#195 · 2 comments ·
All issues in microsoftgraph/msgraph-sdk-php-core
Similar issues
-
jira-created
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 1/5 Under an hour Newbie friendliness 70/100
nunomaduro/phpinsights#745 ·
-
status/awaiting_triage
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
az-digital/az_quickstart#6019 ·