Skip to content

Commit 7831818

Browse files
authored
Merge pull request #38 from nplusai/codex/fix-bug-in-important-codebase-section
Fix Authenticator null check in APIConnector
2 parents 2007748 + 0e0aedd commit 7831818

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

‎SpotifyAPI.Web/Http/APIConnector.cs‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -205,10 +205,10 @@ private async Task<IResponse> DoRequest(IRequest request)
205205
{
206206
response = await _retryHandler.HandleRetry(request, response, async (newRequest) =>
207207
{
208-
await ApplyAuthenticator(request).ConfigureAwait(false);
209-
var newResponse = await _httpClient.DoRequest(request).ConfigureAwait(false);
208+
await ApplyAuthenticator(newRequest).ConfigureAwait(false);
209+
var newResponse = await _httpClient.DoRequest(newRequest).ConfigureAwait(false);
210210
_httpLogger?.OnResponse(newResponse);
211-
ResponseReceived?.Invoke(this, response);
211+
ResponseReceived?.Invoke(this, newResponse);
212212
return newResponse;
213213
}).ConfigureAwait(false);
214214
}
@@ -219,13 +219,13 @@ private async Task<IResponse> DoRequest(IRequest request)
219219
private async Task ApplyAuthenticator(IRequest request)
220220
{
221221
#if NETSTANDARD2_0
222-
if (_authenticator != null
223-
&& !request.Endpoint.IsAbsoluteUri
224-
|| request.Endpoint.AbsoluteUri.Contains("https://api.spotify.com"))
222+
if (_authenticator != null && (
223+
!request.Endpoint.IsAbsoluteUri ||
224+
request.Endpoint.AbsoluteUri.Contains("https://api.spotify.com")))
225225
#else
226-
if (_authenticator != null
227-
&& !request.Endpoint.IsAbsoluteUri
228-
|| request.Endpoint.AbsoluteUri.Contains("https://api.spotify.com", StringComparison.InvariantCulture))
226+
if (_authenticator != null && (
227+
!request.Endpoint.IsAbsoluteUri ||
228+
request.Endpoint.AbsoluteUri.Contains("https://api.spotify.com", StringComparison.InvariantCulture)))
229229
#endif
230230
{
231231
await _authenticator!.Apply(request, this).ConfigureAwait(false);

0 commit comments

Comments
 (0)