Skip to content

[O11y][MongoDB] Update README with support of LDAP authentication #9526

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

Merged
merged 4 commits into from
Apr 16, 2024

Conversation

harnish-crest-data
Copy link
Contributor

  • Enhancement

Proposed commit message

  • Update README with support of LDAP authentication.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Related issues

@harnish-crest-data harnish-crest-data self-assigned this Apr 4, 2024
@harnish-crest-data harnish-crest-data marked this pull request as ready for review April 5, 2024 06:00
@harnish-crest-data harnish-crest-data requested a review from a team as a code owner April 5, 2024 06:00
@elasticmachine
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

Copy link
Contributor

@kush-elastic kush-elastic left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -23,6 +23,18 @@ When using the `directConnection=true` parameter in the connection URI, all oper

The username and password can either be included in the URI or set using the respective configuration options. If included in the URI, these credentials take precedence over any configured username and password configuration options.

Additional URI Support for LDAP Authentication in MongoDB:

- URI: `mongodb://[username:password@]host[:port][/[defaultauthdb][?options]]`
Copy link
Contributor

Choose a reason for hiding this comment

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

Since the URI format mongodb://[username:password@]host[:port][/[defaultauthdb][?options]] has already been specified above in the Complex URI section, we can just include the example for Authentication into the Additional supported URI examples section, like so:

  • Replica set: mongodb://localhost:27017,localhost:27022,localhost:27023/?replicaSet=dbrs
  • Direct connection: mongodb://localhost:27017/?directConnection=true
    When using the directConnection=true parameter in the connection URI, all operations are ..........etc!
  • Authentication: mongodb://username:password@host:port/authSource=$external?authMechanism=PLAIN
    When specifying authMechanism as PLAIN, it indicates .............etc!

@ritalwar
Copy link
Contributor

Are we also updating the beats documentation?

@harnish-crest-data
Copy link
Contributor Author

Are we also updating the beats documentation?

We can update the beats documentation. Let me create an another issue in beats repo for it, thanks!

@harnish-crest-data
Copy link
Contributor Author

Are we also updating the beats documentation?

We can update the beats documentation. Let me create an another issue in beats repo for it, thanks!

Issue: elastic/beats#38963

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @harnish-elastic

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

Copy link
Contributor

@ritalwar ritalwar left a comment

Choose a reason for hiding this comment

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

LGTM!

@harnish-crest-data harnish-crest-data merged commit 563e3ee into elastic:main Apr 16, 2024
@elasticmachine
Copy link

Package mongodb - 1.13.3 containing this change is available at https://epr.elastic.co/search?package=mongodb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[O11y][MongoDB] Analysis on supported and unsupported authentication mechanism and update the documentation
5 participants