Skip to content

Commit

Permalink
Enable nullable
Browse files Browse the repository at this point in the history
  • Loading branch information
mayuki committed Oct 26, 2023
1 parent bd2ba9f commit 5c0fa5c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
<PropertyGroup>
<OutputType>Exe</OutputType>
<TargetFramework>net7.0</TargetFramework>
<Nullable>enable</Nullable>
<ImplicitUsings>enable</ImplicitUsings>
</PropertyGroup>

<ItemGroup>
Expand Down
8 changes: 4 additions & 4 deletions samples/JwtAuthentication/JwtAuthApp.Client/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -124,12 +124,12 @@ public async ValueTask<ResponseContext> SendAsync(RequestContext context, Func<R

AuthenticationTokenStorage.Current.Update(authResult.Token, authResult.Expiration); // NOTE: You can also read the token expiration date from JWT.

context.CallOptions.Headers.Remove(new Metadata.Entry("Authorization", string.Empty));
context.CallOptions.Headers?.Remove(new Metadata.Entry("Authorization", string.Empty));
}

if (!context.CallOptions.Headers.Contains(new Metadata.Entry("Authorization", string.Empty)))
if (!context.CallOptions.Headers?.Contains(new Metadata.Entry("Authorization", string.Empty)) ?? false)
{
context.CallOptions.Headers.Add("Authorization", "Bearer " + AuthenticationTokenStorage.Current.Token);
context.CallOptions.Headers?.Add("Authorization", "Bearer " + AuthenticationTokenStorage.Current.Token);
}

return await next(context);
Expand All @@ -144,7 +144,7 @@ class AuthenticationTokenStorage

private readonly object _syncObject = new object();

public string Token { get; private set; }
public string? Token { get; private set; }
public DateTimeOffset Expiration { get; private set; }

public bool IsExpired => Token == null || Expiration < DateTimeOffset.Now;
Expand Down

0 comments on commit 5c0fa5c

Please sign in to comment.