Skip to content

Commit 0581ed0

Browse files
committed
fix(server): fix setting TTL index on collection : sessions error
1 parent a6c3f1a commit 0581ed0

File tree

5 files changed

+25
-20
lines changed

5 files changed

+25
-20
lines changed

Diff for: app/templates/Gruntfile.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -675,7 +675,7 @@ module.exports = function (grunt) {
675675
setTimeout(function () {
676676
grunt.log.writeln('Done waiting!');
677677
done();
678-
}, 500);
678+
}, 1500);
679679
});
680680

681681
grunt.registerTask('express-keepalive', 'Keep grunt running', function() {
@@ -684,7 +684,7 @@ module.exports = function (grunt) {
684684

685685
grunt.registerTask('serve', function (target) {
686686
if (target === 'dist') {
687-
return grunt.task.run(['build', 'env:all', 'env:prod', 'express:prod', 'open', 'express-keepalive']);
687+
return grunt.task.run(['build', 'env:all', 'env:prod', 'express:prod', 'wait', 'open', 'express-keepalive']);
688688
}
689689

690690
if (target === 'debug') {

Diff for: app/templates/server/app.js

+8-2
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,17 @@ var socketio = require('socket.io').listen(server);
2424
require('./config/socketio')(socketio);<% } %>
2525
require('./config/express')(app);
2626
require('./routes')(app);
27-
27+
<% if (filters.twitterAuth) { %>
28+
mongoose.connection.on('connected', function () {
29+
// Start server
30+
server.listen(config.port, config.ip, function() {
31+
console.log('Express server listening on %d, in %s mode', config.port, app.get('env'));
32+
});
33+
});<% } else { %>
2834
// Start server
2935
server.listen(config.port, config.ip, function () {
3036
console.log('Express server listening on %d, in %s mode', config.port, app.get('env'));
31-
});
37+
});<% } %>
3238

3339
// Expose app
3440
exports = module.exports = app;

Diff for: app/templates/server/config/express.js

+12-14
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@ var path = require('path');
1616
var config = require('./environment');<% if (filters.auth) { %>
1717
var passport = require('passport');<% } %><% if (filters.twitterAuth) { %>
1818
var session = require('express-session');
19-
var mongoStore = require('connect-mongo')(session);<% } %>
19+
var mongoStore = require('connect-mongo')(session);
20+
var mongoose = require('mongoose');<% } %>
2021

2122
module.exports = function(app) {
2223
var env = app.get('env');
@@ -32,19 +33,16 @@ module.exports = function(app) {
3233
app.use(cookieParser());
3334
<% if (filters.auth) { %>app.use(passport.initialize());<% } %><% if (filters.twitterAuth) { %>
3435

35-
// Persist sessions with mongoStore
36-
// We need to enable sessions for passport twitter because its an oauth 1.0 strategy
37-
app.use(session({
38-
secret: config.secrets.session,
39-
resave: true,
40-
saveUninitialized: true,
41-
store: new mongoStore({
42-
url: config.mongo.uri,
43-
collection: 'sessions'
44-
}, function () {
45-
console.log('db connection open' );
46-
})
47-
}));<% } %>
36+
mongoose.connection.on('connected', function () {
37+
// Persist sessions with mongoStore
38+
// We need to enable sessions for passport twitter because its an oauth 1.0 strategy
39+
app.use(session({
40+
secret: config.secrets.session,
41+
resave: true,
42+
saveUninitialized: true,
43+
store: new mongoStore({ mongoose_connection: mongoose.connection })
44+
}));
45+
});<% } %>
4846

4947
if ('production' === env) {
5048
app.use(favicon(path.join(config.root, 'public', 'favicon.ico')));

Diff for: test/fixtures/package.json

+1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
"passport-local": "~0.1.6",
2121
"passport-facebook": "latest",
2222
"passport-twitter": "latest",
23+
"passport-google-oauth": "latest",
2324
"composable-middleware": "^0.3.0",
2425
"connect-mongo": "^0.4.1",
2526
"socket.io": "~0.9.16",

Diff for: test/test-file-creation.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ describe('angular-fullstack generator', function () {
157157
// });
158158
});
159159

160-
describe('with other preprocessors', function() {
160+
describe('with other preprocessors and oauth', function() {
161161
beforeEach(function() {
162162
helpers.mockPrompt(gen, {
163163
script: 'coffee',
@@ -166,7 +166,7 @@ describe('angular-fullstack generator', function () {
166166
router: 'uirouter',
167167
mongoose: true,
168168
auth: true,
169-
oauth: [],
169+
oauth: ['twitterAuth', 'facebookAuth', 'googleAuth'],
170170
socketio: true
171171
});
172172
});

0 commit comments

Comments
 (0)