Opened 16 months ago

Last modified 11 months ago

#30420 assigned Bug

Do not reload runserver during debugging / request processing

Reported by: Daniel Hahler Owned by: Kim Namjun
Component: Core (Management commands) Version: master
Severity: Normal Keywords:
Cc: Florian Apolloner Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

The server should not be restarted while debugging (pdb.set_trace()).

Use case:

  1. pdb.set_trace in the code
  2. trigger web request
  3. start debugging / messing around, maybe even use edit (via pdbpp) to edit files
  4. save something being watched

This ends up in garbarge on the screen, and the server process/thread getting killed.

This could be achieved using a thread lock.

See https://github.com/pallets/werkzeug/pull/1525/files for an example.

Change History (7)

comment:1 Changed 16 months ago by Florian Apolloner

Triage Stage: UnreviewedAccepted

This would indeed be nice to have and looks rather simple to implement. Do you want to submit a PR?

comment:2 Changed 16 months ago by Daniel Hahler

I'd do a PR only after some feedback on https://github.com/pallets/werkzeug/pull/1525.

comment:3 Changed 16 months ago by Florian Apolloner

Cc: Florian Apolloner added

comment:4 Changed 14 months ago by Giovanni Totaro - aka Vanni

Type: Cleanup/optimizationBug

Any update on this bug?

The server process gets killed even on a cold start (i. e. without any file reloading) when a line with import ipdb; ipdb.set_trace() is hit.

comment:5 Changed 14 months ago by Florian Apolloner

If there were an update the current comments would reflect that (usually at least). So the current status is: Patches welcome :)

comment:6 Changed 11 months ago by Kim Namjun

Has patch: set

comment:7 Changed 11 months ago by felixxm

Owner: changed from nobody to Kim Namjun
Patch needs improvement: set
Status: newassigned
Version: 2.2master
Note: See TracTickets for help on using tickets.
Back to Top