PyPI - matlab-proxy - Versions diffs - 0.23.3__py3-none-any.whl → 0.24.1__py3-none-any.whl - Mend - Supply Chain Defender

matlab-proxy 0.23.3py3-none-any.whl → 0.24.1py3-none-any.whl

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Potentially problematic release.

This version of matlab-proxy might be problematic. Click here for more details.

Files changed (55) hide show

matlab_proxy/app_state.py CHANGED Viewed

@@ -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
@@ -1432,6 +1438,7 @@ class AppState:
                         # Wait for matlab to shutdown gracefully
                         await matlab.wait()
                         assert (
                             matlab.returncode == 0
                         ), "Failed to gracefully shutdown MATLAB via the embedded connector"
@@ -1446,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():
@@ -1518,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 CHANGED Viewed

@@ -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>