-
Notifications
You must be signed in to change notification settings - Fork 25.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bias-free communication #33706
Bias-free communication #33706
Conversation
@@ -41,7 +41,7 @@ When downloading files from a different origin than the app, Cross-Origin Resour | |||
|
|||
## Security considerations | |||
|
|||
Use caution when providing users with the ability to download files from a server. Attackers may execute [Denial of Service (DoS)](/windows-hardware/drivers/ifs/denial-of-service) attacks, [API exploitation attacks](https://developer.mozilla.org/docs/Web/HTML/Element/a#security_and_privacy), or attempt to compromise networks and servers in other ways. | |||
Use caution when providing users with the ability to download files from a server. Cyberattackers may execute [Denial of Service (DoS)](/windows-hardware/drivers/ifs/denial-of-service) attacks, [API exploitation attacks](https://developer.mozilla.org/docs/Web/HTML/Element/a#security_and_privacy), or attempt to compromise networks and servers in other ways. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use caution when providing users with the ability to download files from a server. Cyberattackers may execute [Denial of Service (DoS)](/windows-hardware/drivers/ifs/denial-of-service) attacks, [API exploitation attacks](https://developer.mozilla.org/docs/Web/HTML/Element/a#security_and_privacy), or attempt to compromise networks and servers in other ways. | |
Use caution when providing users with the ability to download files from a server. Cyber attackers may execute [Denial of Service (DoS)](/windows-hardware/drivers/ifs/denial-of-service) attacks, [API exploitation attacks](https://developer.mozilla.org/docs/Web/HTML/Element/a#security_and_privacy), or attempt to compromise networks and servers in other ways. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The guidelines call for one word, no hyphen/no space.
https://learn.microsoft.com/en-us/style-guide/militaristic-language
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very minor and feel free to refuse or accept: I think both "cyberattackers"and "cyber attackers" are used in the development community, but I think "cyber attackers" might be used more often.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we have a choice given how explicit they are in the style manual. We've broken with them a bit here and there, but I think AP and Chicago are in agreement. My Chicago is buried at the moment tho ... in a box somewhere around here from the last move 😆.
@@ -401,7 +401,7 @@ Setting the environment variable for `ASPNETCORE_ENVIRONMENT` is only recommende | |||
|
|||
If an app is capable of responding to requests, obtain request, connection, and additional data from the app using terminal inline middleware. For more information and sample code, see <xref:test/troubleshoot#obtain-data-from-an-app>. | |||
|
|||
### Slow or hanging app (IIS) | |||
### Slow or nonresponding app (IIS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
### Slow or nonresponding app (IIS) | |
### Slow or non-responding app (IIS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not per Chicago.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very minor, take or leave: I think you can go either way technically, but I think non-responding might be a bit more common.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"non-" is listed ...
https://learn.microsoft.com/en-us/style-guide/a-z-word-list-term-collections/n/non
... and we defer to Chicago per MS guidelines ...
https://learn.microsoft.com/en-us/style-guide/grammar/grammar-and-parts-of-speech
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
Fixes #33705
IMO, it's a good i-dear to do this every few years to fix problems. This was on my UE tracking issue for Blazor docs, but it's something that I usually do for the whole repo, given that it's just quick search filter stuff.
Notes ...
Not addressed
There are a few articles that are probably OK on he/his/him/her pronoun things because they refer to specific people with a usually obvious gender. Personally, I'd go with something like "user"/"another user" or "user"/"second user," but these are probably OK. Best to check tho in case anyone wants to update the John/Jane examples: