❓ Почему http.ListenAndServe(":8080", nil) считается плохой практикой

nil означает DefaultServeMux, глобальный роутер. В него может зарегистрировать хендлер любой импортированный пакет, например net/http/pprof делает это в init. Так наружу случайно утекают отладочные ручки.

У такого сервера нет тайм аутов. Правильнее создать свой Server и свой ServeMux.

Правильный вариант:

srv := &http.Server{
  Addr:              ":8080",
  Handler:           mux,
  ReadHeaderTimeout: 5 * time.Second,
  ReadTimeout:       10 * time.Second,
  WriteTimeout:      10 * time.Second,
  IdleTimeout:       120 * time.Second,
}
log.Fatal(srv.ListenAndServe())

🐸 Библиотека Go для собеса