refactor(api): unify auth endpoints under /api, enhance signup
This commit is contained in:
parent
52bb003980
commit
f5f86a2bc7
1 changed files with 23 additions and 11 deletions
|
|
@ -1,19 +1,35 @@
|
||||||
from flask import Blueprint, request, jsonify, session
|
from flask import Blueprint, request, jsonify, session
|
||||||
from services.UserService.user import UserService
|
from services.UserService.user import UserService
|
||||||
|
|
||||||
auth_bp = Blueprint("auth", __name__, url_prefix="/auth")
|
auth_bp = Blueprint("auth", __name__, url_prefix="/api")
|
||||||
user_service = UserService()
|
user_service = UserService()
|
||||||
|
|
||||||
|
|
||||||
@auth_bp.route("/signup", methods=["POST"])
|
@auth_bp.route("/signup", methods=["POST"])
|
||||||
def signup():
|
def signup():
|
||||||
data = request.get_json()
|
data = request.get_json()
|
||||||
|
if not data:
|
||||||
|
return jsonify({"message": "No data provided"}), 400
|
||||||
|
|
||||||
|
required_fields = ['username', 'password']
|
||||||
|
for field in required_fields:
|
||||||
|
if not data.get(field):
|
||||||
|
return jsonify({"message": f"{field} is required"}), 400
|
||||||
|
|
||||||
try:
|
try:
|
||||||
new_user = user_service.create_user(data["username"], data["password"])
|
new_user = user_service.create_user(
|
||||||
return (
|
username=data["username"],
|
||||||
jsonify({"message": "User created successfully", "username": new_user.username}),
|
password=data["password"],
|
||||||
201,
|
email=data.get("email"),
|
||||||
|
first_name=data.get("first_name"),
|
||||||
|
last_name=data.get("last_name")
|
||||||
)
|
)
|
||||||
|
|
||||||
|
return jsonify({
|
||||||
|
"message": "User created successfully",
|
||||||
|
"username": new_user.username,
|
||||||
|
"user_id": new_user.id
|
||||||
|
}), 201
|
||||||
|
|
||||||
except ValueError as e:
|
except ValueError as e:
|
||||||
return jsonify({"message": str(e)}), 400
|
return jsonify({"message": str(e)}), 400
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
|
|
@ -21,15 +37,12 @@ def signup():
|
||||||
print(f"Signup error: {e}")
|
print(f"Signup error: {e}")
|
||||||
return jsonify({"message": "Internal server error"}), 500
|
return jsonify({"message": "Internal server error"}), 500
|
||||||
|
|
||||||
|
|
||||||
@auth_bp.route("/login", methods=["POST"])
|
@auth_bp.route("/login", methods=["POST"])
|
||||||
def login():
|
def login():
|
||||||
data = request.get_json()
|
data = request.get_json()
|
||||||
username = data.get("username")
|
username = data.get("username")
|
||||||
password = data.get("password")
|
password = data.get("password")
|
||||||
|
|
||||||
print(f"Login attempt: username={username}, password={password}")
|
print(f"Login attempt: username={username}, password={password}")
|
||||||
|
|
||||||
try:
|
try:
|
||||||
user = user_service.verify_user(username, password)
|
user = user_service.verify_user(username, password)
|
||||||
session["user_id"] = user.id
|
session["user_id"] = user.id
|
||||||
|
|
@ -41,8 +54,7 @@ def login():
|
||||||
print(f"Login error: {e}")
|
print(f"Login error: {e}")
|
||||||
return jsonify({"error": "Internal server error"}), 500
|
return jsonify({"error": "Internal server error"}), 500
|
||||||
|
|
||||||
|
|
||||||
@auth_bp.route("/logout", methods=["POST"])
|
@auth_bp.route("/logout", methods=["POST"])
|
||||||
def logout():
|
def logout():
|
||||||
session.clear()
|
session.clear()
|
||||||
return jsonify({"message": "Logout successful"}), 200
|
return jsonify({"message": "Logout successful"}), 200
|
||||||
Loading…
Add table
Add a link
Reference in a new issue