search

storj-archived / core

Deprecated. Implementation of the Storj v2 protocol for Node.js.
https://storj.io
Other
396 stars 88 forks source link

[WIP] Include responseTime and reputation if available #756

Closed braydonf closed 6 years ago

braydonf commented 6 years ago

This is a quick fix until we deal with the high levels of software entropy that has accumulated.

There are two models for a "Contact" one that exists here in lib/network/contact, and another one is lib/models/contact of MongoDB models. The two come into confusion at: https://github.com/Storj/service-storage-models/blob/master/lib/models/contact.js#L90 where this one is passed into the MongoDB models version, which often lacks all the properties, making it near impossible to set a default value, without inadvertently reseting the value.

braydonf commented 6 years ago

Closing, I have a better idea to solve this issue.

braydonf commented 6 years ago

New solution: https://github.com/Storj/bridge/pull/554