feat: Added 'Access Denied' screen for unauthorized MCP users (#376)

Co-authored-by: Copilot <[email protected]>

Author: lucasgoral

public/locales/en.json

@@ -523,5 +523,14 @@
     "branchTitle": "Branch",
     "gitRepositoryCreated": "Git Repository created successfully",
     "gitRepositoryCreationFailed": "Failed to create Git Repository: {{error}}"
+  },
+  "mcp": {
+    "authorization": {
+      "accessDenied": {
+        "title": "Access Denied",
+        "details": "You are not authorized to see this Managed Control Plane."
+      },
+      "backToWorkspaces": "Back to Workspaces"
+    }
   }
 }

src/components/ControlPlanes/ControlPlaneCard/ControlPlaneCard.tsx

@@ -100,20 +100,21 @@ export const ControlPlaneCard = ({
                   resourceName={controlPlane.metadata.name}
                   resourceType={'managedcontrolplanes'}
                 />
-                {showWarningBecauseOfDisabledSystemIdentityProvider && (
-                  <Infobox size="sm" variant="warning">
+                {showWarningBecauseOfDisabledSystemIdentityProvider ? (
+                  <Infobox size="sm" variant="warning" noMargin>
                     {t('ConnectButton.unsupportedIdP')}
                   </Infobox>
+                ) : (
+                  <ConnectButton
+                    disabled={!isConnectButtonEnabled}
+                    controlPlaneName={name}
+                    projectName={projectName}
+                    workspaceName={workspace.metadata.name ?? ''}
+                    namespace={controlPlane.status?.access?.namespace ?? ''}
+                    secretName={controlPlane.status?.access?.name ?? ''}
+                    secretKey={controlPlane.status?.access?.key ?? ''}
+                  />
                 )}
-                <ConnectButton
-                  disabled={!isConnectButtonEnabled}
-                  controlPlaneName={name}
-                  projectName={projectName}
-                  workspaceName={workspace.metadata.name ?? ''}
-                  namespace={controlPlane.status?.access?.namespace ?? ''}
-                  secretName={controlPlane.status?.access?.name ?? ''}
-                  secretKey={controlPlane.status?.access?.key ?? ''}
-                />
               </FlexBox>
             </FlexBox>
           </FlexBox>

src/components/Ui/Center/Center.module.css

@@ -0,0 +1,12 @@
+.wrapper {
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  flex-direction: column;
+  width: 100%;
+  height: 100%;
+}
+
+.textAlignCenter {
+  text-align: center;
+}

src/components/Ui/Center/Center.tsx

@@ -0,0 +1,19 @@
+import type { PropsWithChildren, ReactNode, CSSProperties } from 'react';
+import cx from 'clsx';
+import styles from './Center.module.css';
+
+export type CenterProps = PropsWithChildren<{
+  className?: string;
+  style?: CSSProperties;
+  textAlignCenter?: boolean;
+}>;
+
+export const Center = ({ children, className, style, textAlignCenter = true }: CenterProps): ReactNode => {
+  const classes = cx(styles.wrapper, { [styles.textAlignCenter]: textAlignCenter }, className);
+
+  return (
+    <div className={classes} style={style}>
+      {children}
+    </div>
+  );
+};

src/components/Ui/Infobox/Infobox.module.css

@@ -17,6 +17,11 @@
   margin-right: 1rem;
 }
 
+.icon-sm {
+  width: 1.25rem;
+  height: 1.25rem;
+}
+
 .content {
   flex-grow: 1;
   padding-right: 0.5rem;
@@ -29,8 +34,9 @@
 }
 
 .size-sm {
-  padding: 0.75rem 1rem;
-  font-size: 0.875rem;
+  padding: 0.5rem 0.875rem;
+  font-size: 0.75rem;
+  border-radius: 0.5rem;
 }
 
 .size-md {
@@ -67,3 +73,7 @@
   color: var(--sapBackgroundColor);
   line-height: 1.2rem;
 }
+
+.no-margin {
+  margin-bottom: 0;
+}

src/components/Ui/Infobox/Infobox.tsx

@@ -12,6 +12,7 @@ interface LabelProps {
   fullWidth?: boolean;
   className?: string;
   icon?: string;
+  noMargin?: boolean;
 }
 
 const variantIcons = {
@@ -29,6 +30,7 @@ export const Infobox: React.FC<LabelProps> = ({
   fullWidth = false,
   className,
   icon,
+  noMargin = false,
 }) => {
   const infoboxClasses = cx(
     styles.infobox,
@@ -41,6 +43,7 @@ export const Infobox: React.FC<LabelProps> = ({
       [styles['variant-warning']]: variant === 'warning',
       [styles['variant-danger']]: variant === 'danger',
       [styles['full-width']]: fullWidth,
+      [styles['no-margin']]: noMargin,
     },
     className,
   );
@@ -49,7 +52,7 @@ export const Infobox: React.FC<LabelProps> = ({
 
   return (
     <div className={infoboxClasses} id={id}>
-      {iconName && <Icon name={iconName} className={styles.icon} />}
+      {iconName && <Icon name={iconName} className={cx(styles.icon, { [styles['icon-sm']]: size === 'sm' })} />}
       <div className={styles.content}>{children}</div>
     </div>
   );

src/components/Ui/NotFoundBanner/NotFoundBanner.module.css

@@ -6,4 +6,4 @@
 .button {
   margin-inline: auto;
   margin-block: 2rem;
-}
+}

src/components/Ui/NotFoundBanner/NotFoundBanner.tsx

@@ -5,6 +5,7 @@ import { Trans, useTranslation } from 'react-i18next';
 import styles from './NotFoundBanner.module.css';
 import { Button } from '@ui5/webcomponents-react';
 import { useNavigate } from 'react-router-dom';
+import { Center } from '../Center/Center.tsx';
 
 export interface NotFoundBannerProps {
   entityType: string;
@@ -14,19 +15,21 @@ export function NotFoundBanner({ entityType }: NotFoundBannerProps) {
   const navigate = useNavigate();
 
   return (
-    <IllustratedBanner
-      illustrationName={IllustrationMessageType.PageNotFound}
-      title={t('NotFoundBanner.titleMessage', { entityType })}
-      subtitle={
-        <div className={styles.subtitleContainer}>
-          <span>
-            <Trans i18nKey="NotFoundBanner.subtitleMessage" values={{ entityType }} />
-          </span>
-          <Button className={styles.button} onClick={() => navigate('/')}>
-            {t('NotFoundBanner.navigateHome')}
-          </Button>
-        </div>
-      }
-    />
+    <Center>
+      <IllustratedBanner
+        illustrationName={IllustrationMessageType.PageNotFound}
+        title={t('NotFoundBanner.titleMessage', { entityType })}
+        subtitle={
+          <div className={styles.subtitleContainer}>
+            <span>
+              <Trans i18nKey="NotFoundBanner.subtitleMessage" values={{ entityType }} />
+            </span>
+            <Button className={styles.button} onClick={() => navigate('/')}>
+              {t('NotFoundBanner.navigateHome')}
+            </Button>
+          </div>
+        }
+      />
+    </Center>
   );
 }

src/lib/api/types/crossplane/CRDList.ts

@@ -40,3 +40,8 @@ export type CRDResponse = {
 export const CRDRequest: Resource<CRDResponse> = {
   path: '/apis/apiextensions.k8s.io/v1/customresourcedefinitions',
 };
+
+export const CRDRequestAuthCheck: Resource<CRDResponse> = {
+  path: '/apis/apiextensions.k8s.io/v1/customresourcedefinitions',
+  jq: '{kind: .kind}',
+};

src/lib/shared/McpContext.tsx

@@ -10,7 +10,6 @@ interface Mcp {
   project: string;
   workspace: string;
   name: string;
-
   secretNamespace?: string;
   secretName?: string;
   secretKey?: string;