Skip to content

Commit

Permalink
Fix config fetcher-related error logging (#61)
Browse files Browse the repository at this point in the history
* Fixes error logging in HttpConfigFetcher.FetchInternalAsync

* Updates CHANGELOG.md
  • Loading branch information
adams85 authored Jan 11, 2023
1 parent a9a2210 commit a31fcfa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
FlagOverrides LocalDictionary(IDictionary<string, object> dictionary, bool watchChanges, OverrideBehaviour overrideBehaviour)
```
Where the `watchChanges` parameter indicates whether the SDK should rebuild the overrides upon each read to keep track of the source dictionary's changes.
- Fix config fetcher-related error logging to include exception in the log if any.

### 7.1.0

Expand Down
2 changes: 1 addition & 1 deletion src/ConfigCatClient/HttpConfigFetcher.cs
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ private async ValueTask<FetchResult> FetchInternalAsync(ProjectConfig lastConfig
errorException = ex;
}

this.log.Error(errorMessage);
this.log.Error(errorMessage, errorException);
this.ReInitializeHttpClient();
return FetchResult.Failure(lastConfig, errorMessage, errorException);
}
Expand Down

0 comments on commit a31fcfa

Please sign in to comment.