diff --git a/src/Antiforgery/src/IAntiforgeryAdditionalDataProvider.cs b/src/Antiforgery/src/IAntiforgeryAdditionalDataProvider.cs index d66b6245db..485fc8c9e8 100644 --- a/src/Antiforgery/src/IAntiforgeryAdditionalDataProvider.cs +++ b/src/Antiforgery/src/IAntiforgeryAdditionalDataProvider.cs @@ -8,7 +8,7 @@ namespace Microsoft.AspNetCore.Antiforgery /// /// Allows providing or validating additional custom data for antiforgery tokens. /// For example, the developer could use this to supply a nonce when the token is - /// generated, then he could validate the nonce when the token is validated. + /// generated, then validate it when the token is validated. /// /// /// The antiforgery system already embeds the client's username within the diff --git a/src/DataProtection/DataProtection/src/AuthenticatedEncryption/IOptimizedAuthenticatedEncryptor.cs b/src/DataProtection/DataProtection/src/AuthenticatedEncryption/IOptimizedAuthenticatedEncryptor.cs index 3cc0a7ca92..9d28c11221 100644 --- a/src/DataProtection/DataProtection/src/AuthenticatedEncryption/IOptimizedAuthenticatedEncryptor.cs +++ b/src/DataProtection/DataProtection/src/AuthenticatedEncryption/IOptimizedAuthenticatedEncryptor.cs @@ -31,10 +31,10 @@ namespace Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption /// This method allows for a slight performance improvement over IAuthenticatedEncryptor.Encrypt /// in the case where the caller needs to prepend or append some data to the resulting ciphertext. /// For instance, if the caller needs to append a 32-bit header to the resulting ciphertext, then - /// he can specify 4 for 'preBufferSize' and overwrite the first 32 bits of the buffer returned + /// specify 4 for 'preBufferSize' and overwrite the first 32 bits of the buffer returned /// by this function. This saves the caller from having to allocate a new buffer to hold the final /// transformed result. - /// + /// /// All cryptography-related exceptions should be homogenized to CryptographicException. /// byte[] Encrypt(ArraySegment plaintext, ArraySegment additionalAuthenticatedData, uint preBufferSize, uint postBufferSize); diff --git a/src/DataProtection/DataProtection/src/IDataProtectionBuilder.cs b/src/DataProtection/DataProtection/src/IDataProtectionBuilder.cs index 95c7c61f50..68db854e74 100644 --- a/src/DataProtection/DataProtection/src/IDataProtectionBuilder.cs +++ b/src/DataProtection/DataProtection/src/IDataProtectionBuilder.cs @@ -15,14 +15,14 @@ namespace Microsoft.AspNetCore.DataProtection /// /// /// If the developer changes the at-rest key protection mechanism, it is intended that - /// he also change the key storage location, and vice versa. For instance, a call to + /// they also change the key storage location, and vice versa. For instance, a call to /// should generally be accompanied by /// a call to , or exceptions may /// occur at runtime due to the data protection system not knowing where to persist keys. /// /// /// Similarly, when a developer modifies the default protected payload cryptographic - /// algorithms, it is intended that he also select an explitiy key storage location. + /// algorithms, they should also set an explicit key storage location. /// A call to /// should therefore generally be paired with a call to , /// for example. @@ -40,4 +40,4 @@ namespace Microsoft.AspNetCore.DataProtection /// IServiceCollection Services { get; } } -} \ No newline at end of file +} diff --git a/src/DataProtection/DataProtection/src/KeyManagement/KeyRingBasedDataProtector.cs b/src/DataProtection/DataProtection/src/KeyManagement/KeyRingBasedDataProtector.cs index d1857d76d2..9fd73d6125 100644 --- a/src/DataProtection/DataProtection/src/KeyManagement/KeyRingBasedDataProtector.cs +++ b/src/DataProtection/DataProtection/src/KeyManagement/KeyRingBasedDataProtector.cs @@ -253,7 +253,7 @@ namespace Microsoft.AspNetCore.DataProtection.KeyManagement } } - // Do we need to notify the caller that he should reprotect the data? + // Do we need to notify the caller that they should reprotect the data? status = UnprotectStatus.Ok; if (keyIdFromPayload != currentKeyRing.DefaultKeyId) { diff --git a/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs b/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs index 8f3fa1904d..badea8559c 100644 --- a/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs +++ b/src/DataProtection/DataProtection/src/KeyManagement/KeyRingProvider.cs @@ -226,9 +226,9 @@ namespace Microsoft.AspNetCore.DataProtection.KeyManagement Volatile.Write(ref _cacheableKeyRing, existingCacheableKeyRing.WithTemporaryExtendedLifetime(utcNow)); } - // The immediate caller should fail so that he can report the error up his chain. This makes it more likely + // The immediate caller should fail so that they can report the error up the chain. This makes it more likely // that an administrator can see the error and react to it as appropriate. The caller can retry the operation - // and will probably have success as long as he falls within the temporary extension mentioned above. + // and will probably have success as long as they fall within the temporary extension mentioned above. throw; } diff --git a/src/Features/JsonPatch/src/Adapters/IObjectAdapter.cs b/src/Features/JsonPatch/src/Adapters/IObjectAdapter.cs index e5206bfa0d..0c15ce0ae6 100644 --- a/src/Features/JsonPatch/src/Adapters/IObjectAdapter.cs +++ b/src/Features/JsonPatch/src/Adapters/IObjectAdapter.cs @@ -7,18 +7,18 @@ namespace Microsoft.AspNetCore.JsonPatch.Adapters { /// /// Defines the operations that can be performed on a JSON patch document. - /// + /// public interface IObjectAdapter { /// /// Using the "add" operation a new value is inserted into the root of the target /// document, into the target array at the specified valid index, or to a target object at /// the specified location. - /// - /// When adding to arrays, the specified index MUST NOT be greater than the number of elements in the array. + /// + /// When adding to arrays, the specified index MUST NOT be greater than the number of elements in the array. /// To append the value to the array, the index of "-" character is used (see [RFC6901]). - /// - /// When adding to an object, if an object member does not already exist, a new member is added to the object at the + /// + /// When adding to an object, if an object member does not already exist, a new member is added to the object at the /// specified location or if an object member does exist, that member's value is replaced. /// /// The operation object MUST contain a "value" member whose content @@ -46,7 +46,7 @@ namespace Microsoft.AspNetCore.JsonPatch.Adapters /// For example: /// /// { "op": "copy", "from": "/a/b/c", "path": "/a/b/e" } - /// + /// /// See RFC 6902 https://tools.ietf.org/html/rfc6902#page-7 /// /// The copy operation. @@ -67,7 +67,7 @@ namespace Microsoft.AspNetCore.JsonPatch.Adapters /// { "op": "move", "from": "/a/b/c", "path": "/a/b/d" } /// /// A location cannot be moved into one of its children. - /// + /// /// See RFC 6902 https://tools.ietf.org/html/rfc6902#page-6 /// /// The move operation. @@ -85,7 +85,7 @@ namespace Microsoft.AspNetCore.JsonPatch.Adapters /// /// If removing an element from an array, any elements above the /// specified index are shifted one position to the left. - /// + /// /// See RFC 6902 https://tools.ietf.org/html/rfc6902#page-6 /// /// The remove operation. @@ -93,7 +93,7 @@ namespace Microsoft.AspNetCore.JsonPatch.Adapters void Remove(Operation operation, object objectToApplyTo); /// - /// Using the "replace" operation he value at the target location is replaced + /// Using the "replace" operation the value at the target location is replaced /// with a new value. The operation object MUST contain a "value" member /// which specifies the replacement value. /// @@ -102,7 +102,7 @@ namespace Microsoft.AspNetCore.JsonPatch.Adapters /// For example: /// /// { "op": "replace", "path": "/a/b/c", "value": 42 } - /// + /// /// See RFC 6902 https://tools.ietf.org/html/rfc6902#page-6 /// /// The replace operation. diff --git a/src/Http/Headers/src/EntityTagHeaderValue.cs b/src/Http/Headers/src/EntityTagHeaderValue.cs index e46cee3a34..9b3a37136b 100644 --- a/src/Http/Headers/src/EntityTagHeaderValue.cs +++ b/src/Http/Headers/src/EntityTagHeaderValue.cs @@ -53,7 +53,7 @@ namespace Microsoft.Net.Http.Headers (length != tag.Length)) { // Note that we don't allow 'W/' prefixes for weak ETags in the 'tag' parameter. If the user wants to - // add a weak ETag, he can set 'isWeak' to true. + // add a weak ETag, they can set 'isWeak' to true. throw new FormatException("Invalid ETag name"); } diff --git a/src/Http/Headers/src/HeaderUtilities.cs b/src/Http/Headers/src/HeaderUtilities.cs index 0d193b4e61..54034359fb 100644 --- a/src/Http/Headers/src/HeaderUtilities.cs +++ b/src/Http/Headers/src/HeaderUtilities.cs @@ -26,7 +26,7 @@ namespace Microsoft.Net.Http.Headers { // Note that even if we check the value here, we can't prevent a user from adding an invalid quality // value using Parameters.Add(). Even if we would prevent the user from adding an invalid value - // using Parameters.Add() he could always add invalid values using HttpHeaders.AddWithoutValidation(). + // using Parameters.Add() they could always add invalid values using HttpHeaders.AddWithoutValidation(). // So this check is really for convenience to show users that they're trying to add an invalid // value. if ((value < 0) || (value > 1)) diff --git a/src/Http/Routing/src/Constraints/HttpMethodRouteConstraint.cs b/src/Http/Routing/src/Constraints/HttpMethodRouteConstraint.cs index 18fbec0c19..ea20e81419 100644 --- a/src/Http/Routing/src/Constraints/HttpMethodRouteConstraint.cs +++ b/src/Http/Routing/src/Constraints/HttpMethodRouteConstraint.cs @@ -73,7 +73,7 @@ namespace Microsoft.AspNetCore.Routing.Constraints // the HTTP GET-specific route will be used for URI generation, which might have undesired behavior. // // To prevent this, a user might call GetVirtualPath(..., { httpMethod = "POST" }) to - // signal that he is generating a URI that will be used for an HTTP POST, so he wants the URI + // signal that they are generating a URI that will be used for an HTTP POST, so they want the URI // generation to be performed by the (b) route instead of the (a) route, consistent with what would // happen on incoming requests. if (!values.TryGetValue(routeKey, out var obj)) diff --git a/src/Identity/Extensions.Core/src/UserManager.cs b/src/Identity/Extensions.Core/src/UserManager.cs index 4116045c4a..e3df5c7734 100644 --- a/src/Identity/Extensions.Core/src/UserManager.cs +++ b/src/Identity/Extensions.Core/src/UserManager.cs @@ -113,7 +113,7 @@ namespace Microsoft.AspNetCore.Identity RegisterTokenProvider(providerName, provider); } } - } + } if (Options.Stores.ProtectPersonalData) { @@ -161,7 +161,7 @@ namespace Microsoft.AspNetCore.Identity /// The used to normalize things like user and role names. /// public ILookupNormalizer KeyNormalizer { get; set; } - + /// /// The used to generate error messages. /// @@ -617,7 +617,7 @@ namespace Microsoft.AspNetCore.Identity /// A normalized value representing the specified . public virtual string NormalizeEmail(string email) => (KeyNormalizer == null) ? email : KeyNormalizer.NormalizeEmail(email); - + private string ProtectPersonalData(string data) { if (Options.Stores.ProtectPersonalData) @@ -861,7 +861,7 @@ namespace Microsoft.AspNetCore.Identity throw new ArgumentNullException(nameof(user)); } var stamp = await securityStore.GetSecurityStampAsync(user, CancellationToken); - if (stamp == null) + if (stamp == null) { Logger.LogWarning(15, "GetSecurityStampAsync for user {userId} failed because stamp was null.", await GetUserIdAsync(user)); throw new InvalidOperationException(Resources.NullSecurityStamp); @@ -1898,7 +1898,7 @@ namespace Microsoft.AspNetCore.Identity } /// - /// Returns a flag indicating whether the specified his locked out, + /// Returns a flag indicating whether the specified is locked out, /// as an asynchronous operation. /// /// The user whose locked out status should be retrieved. diff --git a/src/Mvc/Mvc.Core/src/ApplicationModels/ApplicationModelFactory.cs b/src/Mvc/Mvc.Core/src/ApplicationModels/ApplicationModelFactory.cs index fae18d4481..b86e5465d9 100644 --- a/src/Mvc/Mvc.Core/src/ApplicationModels/ApplicationModelFactory.cs +++ b/src/Mvc/Mvc.Core/src/ApplicationModels/ApplicationModelFactory.cs @@ -131,9 +131,9 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels } private static void ReplaceAttributeRouteTokens( - ControllerModel controller, - ActionModel action, - SelectorModel selector, + ControllerModel controller, + ActionModel action, + SelectorModel selector, List errors) { if (selector.AttributeRouteModel == null) @@ -148,7 +148,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels { "action", action.ActionName }, { "controller", controller.ControllerName }, }; - + foreach (var kvp in action.RouteValues) { routeValues.TryAdd(kvp.Key, kvp.Value); @@ -186,7 +186,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels } private static void AddActionToMethodInfoMap( - Dictionary> actionsByMethod, + Dictionary> actionsByMethod, ActionModel action, SelectorModel selector) { @@ -242,7 +242,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels // we compare it against the rest of the templates that have that same name // associated. // The moment we find one that is different we report the whole group to the - // user in the error message so that he can see the different actions and the + // user in the error message so that they can see the different actions and the // different templates for a given named attribute route. var template = actions[0].selector.AttributeRouteModel.Template; diff --git a/src/ProjectTemplates/Web.Spa.ProjectTemplates/content/React-CSharp/ClientApp/src/components/api-authorization/Login.js b/src/ProjectTemplates/Web.Spa.ProjectTemplates/content/React-CSharp/ClientApp/src/components/api-authorization/Login.js index 7ef201c91c..3954657f30 100644 --- a/src/ProjectTemplates/Web.Spa.ProjectTemplates/content/React-CSharp/ClientApp/src/components/api-authorization/Login.js +++ b/src/ProjectTemplates/Web.Spa.ProjectTemplates/content/React-CSharp/ClientApp/src/components/api-authorization/Login.js @@ -120,7 +120,7 @@ export class Login extends Component { redirectToApiAuthorizationPath(apiAuthorizationPath) { const redirectUrl = `${window.location.origin}${apiAuthorizationPath}`; // It's important that we do a replace here so that when the user hits the back arrow on the - // browser he gets sent back to where it was on the app instead of to an endpoint on this + // browser they get sent back to where it was on the app instead of to an endpoint on this // component. window.location.replace(redirectUrl); }