Skip to content
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

Reference conceptual docs for examples #26759

Merged
merged 4 commits into from
Nov 20, 2021
Merged

Reference conceptual docs for examples #26759

merged 4 commits into from
Nov 20, 2021

Conversation

ajcvickers
Copy link
Contributor

Fixes #19987
Fixes #19987

@ajcvickers ajcvickers requested a review from a team November 19, 2021 12:06
/// <para>
/// However, <b>never</b> pass a concatenated or interpolated string (<c>$""</c>) with non-validated user-provided values
/// into this method. Doing so may expose your application to SQL injection attacks. To use the interpolated string syntax,
/// consider using <see cref="ExecuteSqlInterpolated" /> to create parameters.
/// </para>
/// <para>
/// See <see href="https://aka.ms/efcore-docs-efcore-docs-raw-sql">Executing raw SQL commands with EF Core</see>
/// for more information.
/// for examples and more information.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proposal: Update all links to conceptual docs to be explicit that this is where to find examples. If there is agreement, I will do this in a separate commit.

@ajcvickers
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ajcvickers ajcvickers force-pushed the DeleteAllTheCodes1119 branch from 85bb511 to 0ce648e Compare November 20, 2021 09:23
@ajcvickers ajcvickers merged commit 9f416d6 into main Nov 20, 2021
@ajcvickers ajcvickers deleted the DeleteAllTheCodes1119 branch November 20, 2021 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update raw SQL docs to point to conceptual documentation showing examples
2 participants