Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Allow patchlevel and minor updates to winston-mongodb #812

Merged
merged 1 commit into from
Mar 10, 2017

Conversation

sergiitk
Copy link
Contributor

@sergiitk sergiitk commented Mar 10, 2017

What's this PR do?

Allows patch and minor updates to winston-mongodb, see https://docs.npmjs.com/misc/semver#caret-ranges-123-025-004

How should this be reviewed?

npm install

Any background context you want to provide?

May have caused recent crashes on Gambit prod with the following stacktrace:

MongoError: index not found with name [timestamp_1]
at Function.MongoError.create (/app/node_modules/mongodb-core/lib/error.js:31:11)
at /app/node_modules/mongodb-core/lib/connection/pool.js:483:72
at authenticateStragglers (/app/node_modules/mongodb-core/lib/connection/pool.js:429:16)
at Connection.messageHandler (/app/node_modules/mongodb-core/lib/connection/pool.js:463:5)
at Socket.<anonymous> (/app/node_modules/mongodb-core/lib/connection/connection.js:309:22)
at emitOne (events.js:96:13)
at Socket.emit (events.js:191:7)
at readableAddChunk (_stream_readable.js:176:18)
at Socket.Readable.push (_stream_readable.js:134:10)
at TCP.onread (net.js:563:20)

The problem is described in winstonjs/winston-mongodb#75 and fixed in winston-mongodb@2.0.1. As of today, the most recent version is winston-mongodb@2.0.8.

cc @rapala61

@sergiitk sergiitk changed the title Allow patchlevel updates to winston-mongodb Allow patchlevel and minor updates to winston-mongodb Mar 10, 2017
@aaronschachter aaronschachter merged commit c824f53 into develop Mar 10, 2017
@sergiitk sergiitk deleted the fix-winston-mongo branch March 10, 2017 15:27
@sergiitk
Copy link
Contributor Author

sergiitk commented Mar 10, 2017

A part of #774

@sergiitk
Copy link
Contributor Author

This was an attempt to fix #811

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants