thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (THRIFT-4403) thrift.js: Incorrect usage of 'this' in TWebSocketTransport.__onOpen
Date Mon, 04 Dec 2017 15:05:01 GMT

    [ https://issues.apache.org/jira/browse/THRIFT-4403?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16276906#comment-16276906
] 

ASF GitHub Bot commented on THRIFT-4403:
----------------------------------------

Github user jeking3 commented on a diff in the pull request:

    https://github.com/apache/thrift/pull/1437#discussion_r154672797
  
    --- Diff: lib/js/src/thrift.js ---
    @@ -599,8 +599,8 @@ Thrift.TWebSocketTransport.prototype = {
               //If the user made calls before the connection was fully
               //open, send them now
               this.send_pending.forEach(function(elem) {
    -             this.socket.send(elem.buf);
    -             this.callbacks.push((function() {
    +             self.socket.send(elem.buf);
    --- End diff --
    
    I'm running through cross tests to see if this resolves anything we know failed in the
past.


> thrift.js: Incorrect usage of 'this' in TWebSocketTransport.__onOpen
> --------------------------------------------------------------------
>
>                 Key: THRIFT-4403
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4403
>             Project: Thrift
>          Issue Type: Bug
>          Components: JavaScript - Library
>            Reporter: Philip Frank
>
> Inside the forEach callback 'this' will be the global object, not the transport object,
as the implementation seems to assume.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message