Skip to content

Commit

Permalink
fix(session): flow frames with no handle are not errors
Browse files Browse the repository at this point in the history
As indicated by the spec, a flow frame may be received without a
handle. This indicates a flow frame that should update the session
flow options only, so we shouldn't be printing out warning in that
case.
  • Loading branch information
mbroadst committed Oct 21, 2015
1 parent 5a718a3 commit 505e803
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions lib/session.js
Expand Up @@ -333,9 +333,6 @@ Session.prototype._processFrame = function(frame) {
if (frame.handle !== null) {
if (this._linksByRemoteHandle[frame.handle]) {
this._linksByRemoteHandle[frame.handle]._flowReceived(frame);
} else {
// @todo Proper error reporting. Should we shut down session?
console.warn('received Flow for unknown link(' + frame.handle + '): ' + JSON.stringify(frame));
}
} else {
_.values(this._senderLinks).forEach(function (senderLink) {
Expand Down

0 comments on commit 505e803

Please sign in to comment.