ReactiveX / IxJS

The Interactive Extensions for JavaScript
https://reactivex.io/IxJS/
MIT License
1.32k stars 73 forks source link

async timeout operator does not pass through its input #325

Closed andreasdamm closed 3 years ago

andreasdamm commented 3 years ago

IxJS version: 4.3.1

Code to reproduce: `const {of, first} = require('ix/asynciterable') const {timeout} = require('ix/asynciterable/operators')

async function main() { const value = await first(of(42).pipe(timeout(100))) console.log(value) }

main()`

Expected behavior: output '42' to the console

Actual behavior: outputs 'undefined' to the console

Additional information: commit b0c90cb0798639b2fc878d4e9b5521bfb2bb63d8 introduced this bug by renaming 'value' to 'val' in src/asynciterable/operators/timeout.ts at line 57

trxcllnt commented 3 years ago

Yeah this is wrong. We'll fix it.

trxcllnt commented 3 years ago

Fix released in ix@4.4.0.