ACSTokenGenerator Using DateTime.Now Instead of DateTime.UtcNow

[DaleyKD]

I was looking through the code today to see how I could use an ACS token with PnP.Core. I was looking at the ACSTokenGenerator to try and use it as a base thought on writing an IAuthenticationProvider to cache the token.

I noticed this in there:

pnpframework/src/lib/PnP.Framework/Utilities/ACSTokenGenerator.cs

Lines 32 to 41 in b8413c2

	public string GetToken(Uri siteUrl)
	{
	// implement simple token caching
	if (lastToken == null || DateTime.Now.AddMinutes(3) >= lastToken.ExpiresOn)
	{
	lastToken = tokenHelper.GetAppOnlyAccessToken(TokenHelper.SharePointPrincipal, siteUrl.Authority);
	}
	return lastToken.AccessToken;
	}

Isn't lastToken.ExpiresOn a UTC DateTime?