Skip to content

Fix ArgumentNullException when using minimum json file#41428

Merged
maryamariyan merged 1 commit into
dotnet:masterfrom
maryamariyan:regression
Aug 27, 2020
Merged

Fix ArgumentNullException when using minimum json file#41428
maryamariyan merged 1 commit into
dotnet:masterfrom
maryamariyan:regression

Conversation

@maryamariyan
Copy link
Copy Markdown
Contributor

@ghost
Copy link
Copy Markdown

ghost commented Aug 27, 2020

Tagging subscribers to this area: @maryamariyan
See info in area-owners.md if you want to be subscribed.

@maryamariyan maryamariyan requested review from HaoK and halter73 August 27, 2020 06:18
@alefranz
Copy link
Copy Markdown
Contributor

ouch! sorry about that!

@maryamariyan maryamariyan merged commit 46dd8ed into dotnet:master Aug 27, 2020
@maryamariyan
Copy link
Copy Markdown
Contributor Author

ouch! sorry about that!

No worries.
Thanks @BrennanConroy for reporting the regression.

@maryamariyan maryamariyan deleted the regression branch August 27, 2020 17:33
@maryamariyan
Copy link
Copy Markdown
Contributor Author

/backport to release/5.0

@github-actions
Copy link
Copy Markdown
Contributor

Started backporting to release/5.0: https://github.com/dotnet/runtime/actions/runs/233024330

@BrennanConroy
Copy link
Copy Markdown
Member

Wasn't the original change in 6.0

@github-actions
Copy link
Copy Markdown
Contributor

@maryamariyan backporting to release/5.0 failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Fix ArgumentNullException when using minimum json file
error: sha1 information is lacking or useless (src/libraries/Microsoft.Extensions.Configuration.Json/src/JsonConfigurationFileParser.cs).
error: could not build fake ancestor
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Fix ArgumentNullException when using minimum json file
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ArgumentNullException when using a minimal Json file with the JsonConfigurationProvider

4 participants