Skip to content

feat: added mongodb driver support#116

Merged
hsluoyz merged 1 commit intoapache:masterfrom
ali-shokoohi:feat/add-mongo-driver-support
Oct 1, 2024
Merged

feat: added mongodb driver support#116
hsluoyz merged 1 commit intoapache:masterfrom
ali-shokoohi:feat/add-mongo-driver-support

Conversation

@ali-shokoohi
Copy link
Copy Markdown
Contributor

Added mongodb driver support for connection to a mongodb database as an adapter handler

Changes:

  • Added mongodb-adapter module for connecting to a MongoDB database.
  • Included 'mongodb' as a supported driver.
  • Updated driver switch-case to create a new MongoDB handler based on the provided in.DriverName.
  • Added a configuration example for MongoDB connection in the config/ directory.

Added mongodb driver support for connection to a mongodb database as an adapter handler

Changes:
- Added `mongodb-adapter` module for connecting to a MongoDB database.
- Included 'mongodb' as a supported driver.
- Updated driver switch-case to create a new MongoDB handler based on the provided `in.DriverName`.
- Added a configuration example for MongoDB connection in the `config/` directory.
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 69.072% (-0.7%) from 69.792%
when pulling 1b494a0 on ali-shokoohi:feat/add-mongo-driver-support
into a0bd30b on casbin:master.

@hsluoyz hsluoyz merged commit 3874663 into apache:master Oct 1, 2024
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 1, 2024

🎉 This PR is included in version 1.15.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@hsluoyz
Copy link
Copy Markdown
Member

hsluoyz commented Oct 1, 2024

@ali-shokoohi hi, can you help fix this? #117

@ali-shokoohi
Copy link
Copy Markdown
Contributor Author

@hsluoyz I see. I think we should move RUN go mod download line in the docker file above and remove RUN go get google.golang.org/grpc line. because we have this 'google.golang.org/grpc' package in go.mod.

Let me test it in my local if it works, I'll create a new PR

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.

3 participants