matlab-proxy 0.23.4__py3-none-any.whl → 0.24.1__py3-none-any.whl

matlab_proxy/app_state.py

@@ -1,4 +1,4 @@
-# Copyright 2020-2024 The MathWorks, Inc.
+# Copyright 2020-2025 The MathWorks, Inc.
 
 import asyncio
 import contextlib
@@ -436,8 +436,10 @@ class AppState:
 
             else:
                 await self.matlab_state_updater_lock.release()
-                await self._update_matlab_state_based_on_ready_file_and_connector_status(
-                    matlab_endpoint_to_use
+                await (
+                    self._update_matlab_state_based_on_ready_file_and_connector_status(
+                        matlab_endpoint_to_use
+                    )
                 )
                 logger.debug(
                     f"{this_task}: Required processes are ready, Embedded Connector status is '{self.get_matlab_state()}'"
@@ -1243,7 +1245,7 @@ class AppState:
             delay (int): time delay in seconds before retrying the file read operation
         """
         logger.debug(
-            f'updating matlab_port information from {self.matlab_session_files["matlab_ready_file"]}'
+            f"updating matlab_port information from {self.matlab_session_files['matlab_ready_file']}"
         )
         try:
             await asyncio.wait_for(
@@ -1415,6 +1417,10 @@ class AppState:
         waiters = []
         if matlab is not None:
             if system.is_posix() and matlab.returncode is None:
+                # Close the stderr stream to prevent indefinite hanging on it due to a child
+                # process inheriting it, fixes https://github.com/mathworks/matlab-proxy/issues/44
+                self._close_matlab_stderr_stream(matlab)
+
                 # Sending an exit request to the embedded connector takes time.
                 # When MATLAB is in a "starting" state (implies the Embedded connector is not up)
                 # OR
@@ -1427,23 +1433,9 @@ class AppState:
                 else:
                     logger.debug("Sending HTTP request to stop the MATLAB process...")
                     try:
-                        import sys
-
                         # Send HTTP request
                         await self.__send_stop_request_to_matlab()
 
-                        # Close the stderr stream to prevent indefinite hanging on it due to a child
-                        # process inheriting it, fixes https://github.com/mathworks/matlab-proxy/issues/44
-                        stderr_stream = matlab._transport.get_pipe_transport(
-                            sys.stderr.fileno()
-                        )
-                        if stderr_stream:
-                            logger.debug(
-                                "Closing matlab process stderr stream: %s",
-                                stderr_stream,
-                            )
-                            stderr_stream.close()
-
                         # Wait for matlab to shutdown gracefully
                         await matlab.wait()
 
@@ -1461,9 +1453,10 @@ class AppState:
                         try:
                             matlab.terminate()
                             await matlab.wait()
-                        except:
-                            pass
-
+                        except Exception as ex:
+                            logger.debug(
+                                "Received an exception while terminating matlab: %s", ex
+                            )
             else:
                 # In a windows system
                 if system.is_windows() and matlab.is_running():
@@ -1533,6 +1526,23 @@ class AppState:
         self.matlab_port = None
         logger.debug("Completed Shutdown!!!")
 
+    def _close_matlab_stderr_stream(self, matlab):
+        """
+        This method attempts to close the stderr stream associated with the MATLAB process
+        to prevent potential resource leaks. It logs a debug message if the stream is
+        successfully closed.
+
+        Args:
+            matlab: The MATLAB process reference.
+        """
+        stderr_stream = matlab._transport.get_pipe_transport(sys.stderr.fileno())
+        if stderr_stream:
+            logger.debug(
+                "Closing matlab process stderr stream: %s",
+                stderr_stream,
+            )
+            stderr_stream.close()
+
     async def handle_matlab_output(self):
         """Parse MATLAB output from stdout and raise errors if any."""
         matlab = self.processes["matlab"]

matlab_proxy/gui/index.html

@@ -1 +1,20 @@
-<!doctype html><html lang="en"><head><meta charset="utf-8"/><link rel="icon" href="./favicon.ico"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><meta name="description" content="MATLAB"/><meta name="internal_mw_identifier" content="MWI_MATLAB_PROXY_IDENTIFIER"/><link rel="manifest" href="./manifest.json"/><title>MATLAB</title><script defer="defer" src="./static/js/main.ce0f5505.js"></script><link href="./static/css/main.efa05ff9.css" rel="stylesheet"></head><body><noscript>You need to enable JavaScript to run this app.</noscript><div id="root"></div></body></html>
+ <!-- Copyright 2025 The MathWorks, Inc. -->
+
+<!doctype html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <link rel="icon" href="./favicon.ico" />
+    <meta name="viewport" content="width=device-width, initial-scale=1" />
+    <meta name="theme-color" content="#000000" />
+    <meta name="description" content="MATLAB" />
+    <meta name="internal_mw_identifier" content="MWI_MATLAB_PROXY_IDENTIFIER" />
+    <link rel="manifest" href="./manifest.json" />
+    <title>MATLAB</title>
+    <script type="module" crossorigin src="./static/js/index.Cm14Eqsb.js"></script>
+    <link rel="stylesheet" crossorigin href="./static/css/index.BSVLACuY.css">
+  </head>
+  <body>
+    <div id="root"></div>
+  </body>
+</html>