Better HTTP error codes

[machadoug]

It's better to use 401 for lack of permission and 409 if the user already exists
https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401
https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/409

[aspcanada]

This should be a 403

[codespearhead]

@machadoug Can you change it?

[miaohf]

agree, it should be a 403

[menkotoglou]

Seems like a stale PR. Could that be closed now?

[codespearhead]

This branch is way behind main and OP hasn't acted on the suggestions, but the problem still persists:

Should be status_code=409

full-stack-fastapi-template/backend/app/api/routes/users.py

Lines 58 to 62 in a7f4d7d

	if user:
	raise HTTPException(
	status_code=400,
	detail="The user with this email already exists in the system.",
	)

Should be status_code=409

full-stack-fastapi-template/backend/app/api/routes/users.py

Lines 154 to 158 in a7f4d7d

	if user:
	raise HTTPException(
	status_code=400,
	detail="The user with this email already exists in the system",
	)

Should be `status_code=403 (fixed in 541dd75)

full-stack-fastapi-template/backend/app/api/routes/users.py

Lines 174 to 178 in a7f4d7d

	if not current_user.is_superuser:
	raise HTTPException(
	status_code=403,
	detail="The user doesn't have enough privileges",
	)