@xh/hoist 59.3.0 → 59.3.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 59.3.1 - 2023-11-10
+
+### 🐞 Bug Fixes
+
+* Ensure an unauthorized response from a proxy service endpoint does not prompt the user to refresh
+  and log in again on an SSO-enabled application.
+* Revert change to `Panel` which affected where `className` was applied with `modalSupport` enabled
+
 ## 59.3.0 - 2023-11-09
 
 ### 🎁 New Features

package/core/exception/ExceptionHandler.ts

@@ -345,6 +345,7 @@ export class ExceptionHandler {
     // Detect an expired server session for special messaging, but only for requests back to the
     // app's own server on a relative URL (to avoid triggering w/auth failures on remote CORS URLs).
     private sessionExpired(e: HoistException): boolean {
+        if (XH.appSpec.isSSO) return false;
         const {httpStatus, fetchOptions} = e,
             relativeRequest = !fetchOptions?.url?.startsWith('http');
 

package/desktop/appcontainer/ExceptionDialog.ts

@@ -90,5 +90,5 @@ export const dismissButton = hoistCmp.factory<ExceptionDialogModel>(({model}) =>
 });
 
 function isSessionExpired(e) {
-    return e?.httpStatus === 401;
+    return !XH.appSpec.isSSO && e?.httpStatus === 401;
 }

package/desktop/cmp/panel/Panel.ts

@@ -228,19 +228,21 @@ export const [Panel, panel] = hoistCmp.withFactory<PanelProps>({
 
         const useResizeContainer = resizable || collapsible || showSplitter;
 
-        // 5a) For modalSupport, className + testId need additional frame that will follow content
+        // For modalSupport, create additional frame that will follow content to portal and apply
+        // className and testId accordingly
         if (modalSupportModel) {
             item = modalSupport({
                 model: modalSupportModel,
-                item: frame({item, className, testId})
+                item: frame({
+                    item,
+                    className: model.isModal ? className : undefined,
+                    testId: model.isModal ? testId : undefined
+                })
             });
-
-            return useResizeContainer
-                ? resizeContainer({ref, item})
-                : box({ref, item, ...layoutProps});
         }
 
-        // 5b) No modalSupport, className + testId applied directly to parent
+        testId = model.isModal ? undefined : testId; // Only apply testId once
+
         return useResizeContainer
             ? resizeContainer({ref, item, className, testId})
             : box({ref, item, className, testId, ...layoutProps});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "59.3.0",
+  "version": "59.3.1",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",