guawazi/qiankun-fit
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

🐛 fix dynamic append script with excludeAssetsFilter issue (#900)

Browse Source
This commit is contained in:
Kuitos 2020-08-27 16:13:28 +08:00 committed by GitHub
parent 69953e4f15
commit 843b92c342
1 changed files with 5 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/sandbox/patchers/dynamicAppend.ts
View File

@ -112,15 +112,12 @@ function getOverwrittenAppendChildOrInsertBefore(opts: {
case LINK_TAG_NAME: case LINK_TAG_NAME:
case STYLE_TAG_NAME: { case STYLE_TAG_NAME: {
const stylesheetElement: HTMLLinkElement | HTMLStyleElement = newChild as any; const stylesheetElement: HTMLLinkElement | HTMLStyleElement = newChild as any;
if (!invokedByMicroApp) { const { href } = stylesheetElement as HTMLLinkElement;
if (!invokedByMicroApp || (excludeAssetFilter && href && excludeAssetFilter(href))) {
return rawDOMAppendOrInsertBefore.call(this, element, refChild) as T; return rawDOMAppendOrInsertBefore.call(this, element, refChild) as T;
} }
const mountDOM = appWrapperGetter(); const mountDOM = appWrapperGetter();
const { href } = stylesheetElement as HTMLLinkElement;
if (excludeAssetFilter && href && excludeAssetFilter(href)) {
return rawDOMAppendOrInsertBefore.call(mountDOM, element, refChild) as T;
}
if (scopedCSS) { if (scopedCSS) {
css.process(mountDOM, stylesheetElement, appName); css.process(mountDOM, stylesheetElement, appName);
@ -133,18 +130,13 @@ function getOverwrittenAppendChildOrInsertBefore(opts: {
} }
case SCRIPT_TAG_NAME: { case SCRIPT_TAG_NAME: {
if (!invokedByMicroApp) { const { src, text } = element as HTMLScriptElement;
// some script like jsonp maybe not support cors which should't use execScripts
if (!invokedByMicroApp || (excludeAssetFilter && src && excludeAssetFilter(src))) {
return rawDOMAppendOrInsertBefore.call(this, element, refChild) as T; return rawDOMAppendOrInsertBefore.call(this, element, refChild) as T;
} }
const mountDOM = appWrapperGetter(); const mountDOM = appWrapperGetter();
const { src, text } = element as HTMLScriptElement;
// some script like jsonp maybe not support cors which should't use execScripts
if (excludeAssetFilter && src && excludeAssetFilter(src)) {
return rawDOMAppendOrInsertBefore.call(mountDOM, element, refChild) as T;
}
const { fetch } = frameworkConfiguration; const { fetch } = frameworkConfiguration;
const referenceNode = mountDOM.contains(refChild) ? refChild : null; const referenceNode = mountDOM.contains(refChild) ? refChild : null;