From fefe4571502615a0693a91fe4a88c256ab123ef9 Mon Sep 17 00:00:00 2001 From: Ganesh Patil <7030871503ganeshpatil@gmail.com> Date: Wed, 18 Feb 2026 15:56:52 +0530 Subject: [PATCH] fix: correct subprocess handling and HTTP status codes in /library endpoint (fixes #359) --- fri/server/main.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fri/server/main.py b/fri/server/main.py index c2e1e659..42d88561 100644 --- a/fri/server/main.py +++ b/fri/server/main.py @@ -507,18 +507,18 @@ def library(dir): result = subprocess.run(["cmd.exe", "/c", r"..\library.bat", library_path, filename], cwd=dir_path, check=True, capture_output=True, text=True) proc = result.stdout else: - proc = subprocess.check_output([r"../library", library_path, filename], cwd=dir_path) + proc = subprocess.check_output([r"../library", library_path, filename], cwd=dir_path, stderr=subprocess.STDOUT) proc = proc.decode("utf-8") resp = jsonify({'message': proc}) - resp.status_code = 201 + resp.status_code = 200 return resp except subprocess.CalledProcessError as e: error_output = get_error_output(e) resp = jsonify({'message': f'Command execution failed: {error_output}'}) - resp.status_code = 500 + resp.status_code = 400 return resp except Exception as e: - resp = jsonify({'message': 'There is an Error'}) + resp = jsonify({'message': 'Internal server error'}) resp.status_code = 500 return resp