Hi Bence, welcome! 

There are already a couple of tests in place to check the remove fetching: 

https://github.com/django/django/blob/05bbff82638731a6abfed2fe0ae06a4d429cb32f/tests/admin_scripts/tests.py#L2047-L2072

Without changing the command code I'd first add view (in 
admin_scripts/urls.py is fine) to first check for basic auth credentials 
and then pass off to `serve` (as the existing route already does). That 
should give us the reproduce (or we can closed as fixed). 

Then I think it's easier if you open a PR with your suggested change after 
that (but more-or-less sounds plausible without looking in depth.) 

Hopefully that gets you going? 

Kind Regards,

Carlton

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/66771c44-5b9f-4c29-b92a-91cd3092e016n%40googlegroups.com.
  • Hel... Bence Gáspár
    • ... Carlton Gibson
      • ... Bence Gáspár
        • ... Tim Graham
          • ... Bence Gáspár
          • ... 'Adam Johnson' via Django developers (Contributions to Django itself)
            • ... Tom Forbes

Reply via email to