-
Notifications
You must be signed in to change notification settings - Fork 21
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
DOCSP-43148 - Refactor AWS authentication #258
Conversation
✅ Deploy Preview for mongodb-docs-csharp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
The following sections contain code examples that use the following placeholders: | ||
To use AWS authentication, first perform the following steps: | ||
|
||
- Add the ``MongoDB.Driver.Authentication.AWS`` NuGet package to your project. |
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.
Should we provide a link to nuget? https://www.nuget.org/packages/MongoDB.Driver.Authentication.AWS
|
||
.. code-block:: csharp | ||
|
||
MongoAuthentication.RegisterAwsAuthenticationProvider(); |
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.
Replace the code with the following one:
MongoClientSettings.Extensions.AddAWSAuthentication();
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.
That's really weird. I don't know where that line came from.
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.
LGTM
Co-authored-by: Jordan Smith <[email protected]> (cherry picked from commit 1d75584)
Cloned from #250 after inadvertently closing v3.0 branch
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-43148
Staging Links
Self-Review Checklist