• Samuel Pitoiset's avatar
    rtmp: Do not send _checkbw calls as notifications · 8b6a5a79
    Samuel Pitoiset authored
    The _checkbw calls were changed to use transactionId 0 in commit
    82613564 so that servers would not return _result/_error about it.
    While this is the strict interpretation of the spec, there are
    servers that return _error about it, even if transactionId was 0.
    
    The latest version of EvoStream Media Server (the commercial version
    of crtmpserver) behaves properly as described, i.e. returning an
    _error normally but not returning anything when using transactionId
    0. The latest version of crtmpserver (right now at least) doesn't
    behave like this though, it returns an error even if transactionId
    was 0.
    
    There are also other servers that return errors even if transactionId
    is set to 0. Therefore set a proper transaction id so that the invoke
    can be tracked and the error properly ignored instead.
    Signed-off-by: 's avatarMartin Storsjö <martin@martin.st>
    8b6a5a79
rtmp.h 2.29 KB