Skip to content

Fix get user from request blocks loop - #236

Open
bhtabor wants to merge 4 commits into
Promptly-Technologies-LLC:mainfrom
bhtabor:fix/get_user_from_request_blocks_loop
Open

Fix get user from request blocks loop#236
bhtabor wants to merge 4 commits into
Promptly-Technologies-LLC:mainfrom
bhtabor:fix/get_user_from_request_blocks_loop

Conversation

@bhtabor

@bhtabor bhtabor commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Fix get user from request blocks loop

Fix get user from request blocks loop
@chriscarrollsmith

Copy link
Copy Markdown
Contributor

Follow-up from review:

  • Filed Document sync-Session + async-route pattern; add optional AsyncSession dependency #237 to document the sync-Session + rare-async-route pattern and optionally add an AsyncSession dependency.
  • Pushing a follow-up commit on this branch for:
    • safer update_profile (offload CPU-only image work; keep Session/ORM mutations out of the thread pool)
    • convert remaining falsely-async leftovers (read_static_page, read_home)
    • typed _run_async test helper

Offload only CPU-bound image processing, convert remaining falsely-async
routes, and type the async test helper. Follow-up: Promptly-Technologies-LLC#237.
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

Successfully merging this pull request may close these issues.

3 participants