-
Notifications
You must be signed in to change notification settings - Fork 231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
authProviderOptions folder seems to be missing and causing compilation warning #1385
Comments
Can you share a super small repro so we can have a look at it? I wasn't able to reproduce. Thanks! |
Same issue here, in my case, I was trying to use @microsoft/microsoft-graph-client in a react app. @sebastienlevert, here are the repro steps:
import React from 'react';
import logo from './logo.svg';
import './App.css';
import { InteractiveBrowserCredential } from "@azure/identity";
import { Client } from "@microsoft/microsoft-graph-client";
import { TokenCredentialAuthenticationProvider } from "@microsoft/microsoft-graph-client/authProviders/azureTokenCredentials";
function App() {
const credential = new InteractiveBrowserCredential({
clientId: "YOUR_CLIENT_ID",
tenantId: "YOUR_TENANT_ID",
});
const authProvider = new TokenCredentialAuthenticationProvider(credential, {
scopes: ["User.Read"],
});
const client = Client.initWithMiddleware({ authProvider });
return (
<div className="App">
<header className="App-header">
<img src={logo} className="App-logo" alt="logo" />
<p>
Edit <code>src/App.tsx</code> and save to reload.
</p>
<a
className="App-link"
href="https://reactjs.org"
target="_blank"
rel="noopener noreferrer"
>
Learn React
</a>
</header>
</div>
);
}
export default App; |
Thanks @yiqing-zhao for the repro, I was able to get this reproduced. It seems to be an ESM issue that doesn't appear in CJS. @koros please have a look as this is really impacting developers right now. Thanks. |
Thanks @yiqing-zhao. You beat me to it. Things are mad here and I haven't had a chance to create a repro for @sebastienlevert. One less thing on my task list 😊 |
@yiqing-zhao I am currently looking into this. I'll give an update soon |
Hello everyone, Do you have a date when a fix will be provided? |
It should be addressed in ver |
Bug Report
Prerequisites
For more information, see the
CONTRIBUTING
guide.Description
I'm getting the following compilation warning:
WARNING in ./node_modules/@microsoft/microsoft-graph-client/authProviders/authCodeMsalBrowser/index.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from 'C:\Projects\Obedis\src\Frontend\Obedis.Cockpit\node_modules@microsoft\microsoft-graph-client\authProviderOptions\authCodeMsalBrowser\index.ts' file: ErrCompiled with warnings.
I've checked under the node modules folder in the file system and the "authProviderOptions" folder does not exist.
I've re-installed the component but still getting the same error.
Many thanks in advanced
The text was updated successfully, but these errors were encountered: