Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

async timeout operator does not pass through its input #325

Closed
andreasdamm opened this issue Jun 3, 2021 · 2 comments
Closed

async timeout operator does not pass through its input #325

andreasdamm opened this issue Jun 3, 2021 · 2 comments

Comments

@andreasdamm
Copy link

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 b0c90cb introduced this bug by renaming 'value' to 'val' in src/asynciterable/operators/timeout.ts at line 57

@trxcllnt
Copy link
Member

trxcllnt commented Jun 8, 2021

Yeah this is wrong. We'll fix it.

trxcllnt added a commit that referenced this issue Jun 9, 2021
…eout operator passes its values

Typo introduced by linting in a refactor prevented AsyncIterable from passing on its values.

fix #325
@trxcllnt trxcllnt mentioned this issue Jun 9, 2021
@trxcllnt
Copy link
Member

trxcllnt commented Jun 9, 2021

Fix released in ix@4.4.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants