Skip to content

docs(jupyter-notebook): document zmq transport fallback and disable_check_xsrf pitfalls - #70482

Merged
teknium1 merged 2 commits into
mainfrom
hermes/hermes-03e6b270
Jul 24, 2026
Merged

docs(jupyter-notebook): document zmq transport fallback and disable_check_xsrf pitfalls#70482
teknium1 merged 2 commits into
mainfrom
hermes/hermes-03e6b270

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Salvages #27283 by @upicat — documents two easy-to-misdiagnose pitfalls in the jupyter-notebook skill (formerly jupyter-live-kernel, moved to optional-skills in #70431 after the PR was opened).

  • Pitfall 9 — zmq transport fallback: on some hosts the websocket reply channel hangs on every execute even though the kernel ran fine (REST shows execution_state=idle, execution_count increments). Fix: --transport zmq to drive the kernel via jupyter_client directly.
  • Pitfall 10 — disable_check_xsrf for REST-only flows: a fresh ServerApp rejects POST /api/sessions with '_xsrf' argument missing from POST since no browser ever establishes a token. Fix: start with --ServerApp.disable_check_xsrf=True.

Changes

Validation

Check Result
Cherry-pick onto current main clean, applied to renamed path
Scope docs-only, 4 files

Infographic

jupyter-notebook-pitfalls

upicat and others added 2 commits July 23, 2026 19:33
…e_check_xsrf pitfalls

Add two pitfalls discovered when running the skill against a fresh
Jupyter server:

- Pitfall #9: When the websocket reply channel hangs on every execute
  even though the kernel actually ran (REST shows execution_state=idle
  and execution_count increments), force zmq transport with
  --transport zmq. The zmq transport uses jupyter_client directly and
  sidesteps the broken websocket layer.

- Pitfall #10: A fresh ServerApp rejects POST /api/sessions with
  "_xsrf argument missing from POST" unless you start it with
  --ServerApp.disable_check_xsrf=True. Needed for REST-only flows
  where no browser/cookie is establishing the XSRF token.
Regenerate the auto-gen skill page and translate pitfalls 9-10 into the
zh-Hans mirror, following the cherry-picked pitfall additions.
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) labels Jul 24, 2026
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

ran on 1158d50

all good!

@teknium1
teknium1 merged commit df1464e into main Jul 24, 2026
38 checks passed
@teknium1
teknium1 deleted the hermes/hermes-03e6b270 branch July 24, 2026 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants