Merge pull request #54 from pnp/undocked-spshooter-fix

Undocked spshooter fix

Author: tavikukko

gulpfile.js

@@ -224,6 +224,7 @@ gulp.task('copy:sp-taxonomy', (done) => {
 gulp.task('copy:microsoft-graph-types', (done) => {
   console.log("Copy @microsoft/microsoft-graph-types");
   gulp.src('./node_modules/@microsoft/microsoft-graph-types/microsoft-graph.d.ts')
+    .pipe(rename('index.d.ts'))
     .pipe(gulp.dest('./public/@microsoft/microsoft-graph-types/'));
   done();
 });
@@ -326,7 +327,7 @@ gulp.task('default',
    // 'copy:sp-taxonomy',
     'copy:microsoft-graph-types',
     'copy:microsoft-graph-client',
-    'copy:msal',
+   // 'copy:msal',
     'copy:msal-browser',
    // 'copy:AuthCodeMSALBrowserAuthenticationProvider',
     'copy:react',

iframe-sandbox-app/package-lock.json

@@ -5,9 +5,9 @@
   "homepage": ".",
   "dependencies": {
     "@fluentui/react": "^8.104.1",
-    "@microsoft/mgt": "^2.9.0",
-    "@microsoft/mgt-element": "^2.9.0",
-    "@microsoft/mgt-react": "^2.9.0",
+    "@microsoft/mgt": "^3.0.1",
+    "@microsoft/mgt-element": "^3.0.1",
+    "@microsoft/mgt-react": "^3.0.1",
     "@testing-library/jest-dom": "^5.16.5",
     "@testing-library/react": "^13.4.0",
     "@testing-library/user-event": "^13.5.0",

iframe-sandbox-app/package.json

@@ -39,6 +39,23 @@ const MGTIframe = () => {
     // window.removeEventListener("message", onMessageReceivedFromIframe);
   }, [onMessageReceivedFromIframe]);
 
+  React.useEffect(function setupListener() {
+    function handleResize(x: any) {
+      console.log('theme changed', x.detail)
+      if(x.detail){
+        document.body.style.backgroundColor = "black";
+      } else {
+        document.body.style.backgroundColor = "white";
+      }
+    }
+    window.addEventListener('darkmodechanged', handleResize)
+
+    return function cleanupListener() {
+      console.log('removing listener')
+      window.removeEventListener('darkmodechanged', handleResize)
+    }
+  })
+
   CacheService.config.isEnabled = false;
 
   var getAccessToken = async (scopes: any[]): Promise<string> => {

iframe-sandbox-app/src/mgtiframe/MGTIframe.tsx

@@ -1,17 +1,17 @@
 {
   "name": "sp-editor",
-  "version": "6.6.1",
+  "version": "6.6.2",
   "private": true,
   "homepage": ".",
   "dependencies": {
-    "@azure/msal-browser": "^2.37.0",
-    "@azure/msal-react": "^1.5.7",
-    "@fluentui/date-time-utilities": "^8.5.10",
-    "@fluentui/react": "^8.109.2",
+    "@azure/msal-browser": "^2.38.0",
+    "@azure/msal-react": "^1.5.9",
+    "@fluentui/date-time-utilities": "^8.5.13",
+    "@fluentui/react": "^8.110.10",
     "@ionic/react": "^5.8.4",
     "@ionic/react-router": "^5.8.4",
-    "@microsoft/mgt": "^2.10.1",
-    "@microsoft/mgt-react": "^2.10.1",
+    "@microsoft/mgt": "^3.0.1",
+    "@microsoft/mgt-react": "^3.0.1",
     "@microsoft/microsoft-graph-client": "^3.0.5",
     "@minoru/react-dnd-treeview": "^3.4.0",
     "@uifabric/theme-samples": "^7.5.29",
@@ -52,14 +52,14 @@
     ]
   },
   "devDependencies": {
-    "@pnp/core": "^3.14.0",
-    "@pnp/graph": "^3.14.0",
-    "@pnp/logging": "^3.14.0",
-    "@pnp/msaljsclient": "^3.14.0",
-    "@pnp/nodejs": "^3.14.0",
-    "@pnp/queryable": "^3.14.0",
-    "@pnp/sp": "^3.14.0",
-    "@pnp/sp-admin": "^3.14.0",
+    "@pnp/core": "^3.16.0",
+    "@pnp/graph": "^3.16.0",
+    "@pnp/logging": "^3.16.0",
+    "@pnp/msaljsclient": "^3.16.0",
+    "@pnp/nodejs": "^3.16.0",
+    "@pnp/queryable": "^3.16.0",
+    "@pnp/sp": "^3.16.0",
+    "@pnp/sp-admin": "^3.16.0",
     "@types/chrome": "0.0.204",
     "@types/jest": "29.2.4",
     "@types/jscodeshift": "^0.11.6",

package-lock.json

@@ -82,6 +82,10 @@ export declare type CacheOptions = {
      * If set, MSAL will attempt to migrate cache entries from older versions on initialization. By default this flag is set to true if cacheLocation is localStorage, otherwise false.
      */
     cacheMigrationEnabled?: boolean;
+    /**
+     * Flag that determines whether access tokens are stored based on requested claims
+     */
+    claimsBasedCachingEnabled?: boolean;
 };
 export declare type BrowserSystemOptions = SystemOptions & {
     /**

package.json

@@ -22,6 +22,7 @@ export declare enum EventType {
     LOGOUT_START = "msal:logoutStart",
     LOGOUT_SUCCESS = "msal:logoutSuccess",
     LOGOUT_FAILURE = "msal:logoutFailure",
-    LOGOUT_END = "msal:logoutEnd"
+    LOGOUT_END = "msal:logoutEnd",
+    RESTORE_FROM_BFCACHE = "msal:restoreFromBFCache"
 }
 //# sourceMappingURL=EventType.d.ts.map

public/@azure/msal-browser/config/Configuration.d.ts

@@ -27,7 +27,7 @@ export declare abstract class BaseInteractionClient {
      * Initializer function for all request APIs
      * @param request
      */
-    protected initializeBaseRequest(request: Partial<BaseAuthRequest>): Promise<BaseAuthRequest>;
+    protected initializeBaseRequest(request: Partial<BaseAuthRequest>, account?: AccountInfo): Promise<BaseAuthRequest>;
     /**
      *
      * Use to get the redirect uri configured in MSAL or null.
@@ -36,6 +36,7 @@ export declare abstract class BaseInteractionClient {
      *
      */
     getRedirectUri(requestRedirectUri?: string): string;
+    validateRequestAuthority(authority: string, account: AccountInfo): Promise<void>;
     /**
      *
      * @param apiId

public/@azure/msal-browser/event/EventType.d.ts

@@ -121,7 +121,7 @@ export declare class NativeInteractionClient extends BaseInteractionClient {
      * @param tenantId
      * @param reqTimestamp
      */
-    cacheNativeTokens(response: NativeResponse, request: NativeTokenRequest, homeAccountIdentifier: string, idTokenObj: AuthToken, responseAccessToken: string, tenantId: string, reqTimestamp: number): void;
+    cacheNativeTokens(response: NativeResponse, request: NativeTokenRequest, homeAccountIdentifier: string, accountEntity: AccountEntity, idTokenObj: AuthToken, responseAccessToken: string, tenantId: string, reqTimestamp: number): void;
     protected addTelemetryFromNativeResponse(response: NativeResponse): MATS | null;
     /**
      * Validates native platform response before processing