Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: added mongodb driver support #116

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

ali-shokoohi
Copy link
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

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 casbin:master Oct 1, 2024
4 of 5 checks passed
Copy link

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
Member

hsluoyz commented Oct 1, 2024

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

@ali-shokoohi
Copy link
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

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

Successfully merging this pull request may close these issues.

3 participants