Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

added db_options to config objetc #132

Closed
wants to merge 1 commit into from

2 participants

@ericchaves

Hi Folks,

It seems that some new release of mongo driver is nagging about the db_options not being passed. I just added a monkey patch so we can pass db_options as required. feel free to dismiss it if not aligned to restful code organization.

I also noticed that running the tests in mongodb-support-2 does not pass. Since the error is not related to the mongodb engine, I'm assuming this is solved in master.

Is there any plans of merging this branch into master?

Cheers,

Eric

@indexzero indexzero closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 22, 2012
  1. @ericchaves
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 2 deletions.
  1. +1 −1  lib/resourceful/engines/mongodb.js
  2. +2 −1  test/engines/mongodb.js
View
2  lib/resourceful/engines/mongodb.js
@@ -20,7 +20,7 @@ var Mongodb = exports.Mongodb = function Mongodb (config) {
this.db = new Db(config.database || resourceful.env || 'test',
new Server(config.host || config.uri || '127.0.0.1',
- config.port || 27017));
+ config.port || 27017), config.db_options);
this.collection = config.collection || 'resourceful';
View
3  test/engines/mongodb.js
@@ -10,7 +10,8 @@ engine.options = {
database: 'test',
host: '127.0.0.1',
port: 27017,
- collection: 'resourceful'
+ collection: 'resourceful',
+ db_options: {safe:true}
};
engine.load = function (resourceful, data, callback) {
Something went wrong with that request. Please try again.