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

Holding down Enter key triggers "AbortError: signal is aborted without reason" #3124

Open
philrz opened this issue Jul 8, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@philrz
Copy link
Contributor

philrz commented Jul 8, 2024

Repro is with Zui commit 4077181.

This can be considered an "unintentional torture test". 😉 That is, I wasn't trying to trigger a bug, and I happened to be doing something that's unlike what I expect a normal user would be doing. Therefore it's probably not urgent to fix. But I file 'em anyway as I see 'em.

Repro steps as shown in the video below:

  1. Load the sample.zng test data from the Zui repo
  2. Click Query Pool
  3. Hold down the Enter key for several seconds

After I let up on the Enter key, an Unhandled Runtime Error "AbortError: signal is aborted without reason" appears.

Repro.mp4

The complete Source and Call Stack shown:

Source
../packages/zed-js/src/client/utils.ts (78:46) @ abort

  76 | export function wrapAbort(signal?: IsoAbortSignal) {
  77 | const ctl = new AbortController();
> 78 | signal?.addEventListener('abort', () => ctl.abort());
     |                                            ^
  79 | return ctl;
  80 | }
  81 | 
Call Stack
abort
../packages/zed-js/src/client/utils.ts (78:46)
abort
src/modules/async-tasks/async-task.ts (13:20)
abort
src/modules/async-tasks/async-tasks.ts (23:46)
abort
src/modules/async-tasks/async-tasks.ts (18:9)
createOrReplace
src/core/query/run.ts (40:32)
handleIt
src/core/handlers.ts (41:11)
run
src/core/query/run.ts (28:4)
eval
../node_modules/redux-thunk/es/index.js (16:0)
dispatch
src/views/results-pane/run-results-query.tsx (13:4)
handleIt
src/core/handlers.ts (41:11)
runResultsMain
src/views/session-page/handler.ts (52:20)
startTransition
../node_modules/react/cjs/react.development.js (2428:0)
SessionPageHandler.fetchResults
src/views/session-page/handler.ts (51:20)
fetchResults
src/views/session-page/handler.ts (31:9)
load
src/views/session-page/index.tsx (21:31)
commitHookEffectListMount
../node_modules/react-dom/cjs/react-dom.development.js (22969:0)
commitLayoutEffectOnFiber
../node_modules/react-dom/cjs/react-dom.development.js (23081:0)
commitLayoutMountEffects_complete
../node_modules/react-dom/cjs/react-dom.development.js (24461:0)
commitLayoutEffects_begin
../node_modules/react-dom/cjs/react-dom.development.js (24447:0)
commitLayoutEffects
../node_modules/react-dom/cjs/react-dom.development.js (24385:0)
commitRootImpl
../node_modules/react-dom/cjs/react-dom.development.js (26651:0)
commitRoot
../node_modules/react-dom/cjs/react-dom.development.js (26517:0)
performSyncWorkOnRoot
../node_modules/react-dom/cjs/react-dom.development.js (25956:0)
flushSyncCallbacks
../node_modules/react-dom/cjs/react-dom.development.js (11982:0)
eval
../node_modules/react-dom/cjs/react-dom.development.js (25490:0)
@philrz philrz added the bug Something isn't working label Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant