Skip to content

Commit

Permalink
fix(SenderLink): temporarily remove transfersInFlight
Browse files Browse the repository at this point in the history
Session._transfersInFlight was used to keep track of messages that
may have been required to be resent upon link reattachment. As we
currently have no logic for this (and this was being set whether we
were guaranteed to get disposition for the frame or not), I've
opted to remove this until that behavior is more completely
implemented.
  • Loading branch information
mbroadst committed Jul 11, 2015
1 parent cc51396 commit d8738cc
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
6 changes: 0 additions & 6 deletions lib/sender_link.js
Expand Up @@ -122,12 +122,6 @@ SenderLink.prototype._sendMessage = function(message, options) {
}
}

this.session._transfersInFlight[messageId] = {
message: message,
options: options,
sent: Date.now()
};

// create frame and send
var transferOptions = _.defaults(options, {
channel: this.session.channel,
Expand Down
1 change: 0 additions & 1 deletion lib/session.js
Expand Up @@ -120,7 +120,6 @@ function Session(conn) {
this._linksByName = {};
this._linksByRemoteHandle = {};
this._senderLinks = [];
this._transfersInFlight = {};
this._deliveryTag = 1;

var self = this;
Expand Down

0 comments on commit d8738cc

Please sign in to comment.