Simplify code of http_exception_handler

[PipeKnight]

Looking to JSONResponse.__init__() function, you can see that headers parameter is None by default, so it doesn't matter do we pass it here as None or do not pass at all. That's why we can safely remove this if-else expression and slightly simplify the code

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (cf73051) 100.00% compared to head (3cfb1c3) 100.00%.
Report is 749 commits behind head on master.

❗ Current head 3cfb1c3 differs from pull request most recent head 089071a. Consider uploading reports for the commit 089071a to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##            master     #5323    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          540       538     -2     
  Lines        13969     13869   -100     
==========================================
- Hits         13969     13869   -100     

Files	Coverage Δ
fastapi/exception_handlers.py	100.00% <100.00%> (ø)

... and 17 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

📝 Docs preview for commit e9fd538 at: https://630e16343174f0607e6e27a3--fastapi.netlify.app

[github-actions]

📝 Docs preview for commit 3cfb1c3 at: https://631224982328a715fc06019d--fastapi.netlify.app

[iudeen]

LGTM

[ramnes]

LGTM but you probably can close it, it doesn't make sense anymore with the latest implementation of http_exception_handler from #5365:

fastapi/fastapi/exception_handlers.py

Lines 10 to 16 in 97f04ab

	async def http_exception_handler(request: Request, exc: HTTPException) -> Response:
	headers = getattr(exc, "headers", None)
	if not is_body_allowed_for_status_code(exc.status_code):
	return Response(status_code=exc.status_code, headers=headers)
	return JSONResponse(
	{"detail": exc.detail}, status_code=exc.status_code, headers=headers
	)

[tiangolo]

Thanks @PipeKnight! 🍰

And thanks everyone for the reviews and comments.

It seems this was covered in a recent change, so I'll close this one. But thanks for the effort! ☕