Skip to content

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

Merged
harnish-crest-data merged 4 commits into
elastic:mainfrom
chavdaharnish:mongodb-ldap-auth
Apr 16, 2024
Merged

[O11y][MongoDB] Update README with support of LDAP authentication#9526
harnish-crest-data merged 4 commits into
elastic:mainfrom
chavdaharnish:mongodb-ldap-auth

Conversation

@harnish-crest-data

Copy link
Copy Markdown
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
Comment thread packages/mongodb/docs/README.md
@elasticmachine

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@kush-elastic kush-elastic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!


Additional URI Support for LDAP Authentication in MongoDB:

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

Copy link
Copy Markdown
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
Copy Markdown
Contributor

Are we also updating the beats documentation?

@harnish-crest-data

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown

💚 Build Succeeded

History

cc @harnish-elastic

@elastic-sonarqube

Copy link
Copy Markdown

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

@ritalwar ritalwar left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

5 participants