MOBILE-4303 iframe: Fix auto-login applied in H5P local URLs

main
Dani Palou 2023-05-04 11:59:01 +02:00
parent 8ce5414980
commit f8d3c023cc
2 changed files with 16 additions and 10 deletions

View File

@ -156,8 +156,14 @@ export class CoreIframeComponent implements OnChanges, OnDestroy {
* Detect changes on input properties.
*/
async ngOnChanges(changes: {[name: string]: SimpleChange }): Promise<void> {
if (changes.src) {
let url = CoreUrlUtils.getYoutubeEmbedUrl(changes.src.currentValue) || changes.src.currentValue;
if (!changes.src) {
return;
}
let url = changes.src.currentValue;
if (!CoreUrlUtils.isLocalFileUrl(url)) {
url = CoreUrlUtils.getYoutubeEmbedUrl(changes.src.currentValue) || changes.src.currentValue;
this.displayHelp = CoreIframeUtils.shouldDisplayHelpForUrl(url);
const currentSite = CoreSites.getCurrentSite();
@ -173,6 +179,7 @@ export class CoreIframeComponent implements OnChanges, OnDestroy {
}
await CoreIframeUtils.fixIframeCookies(url);
}
this.safeUrl = DomSanitizer.bypassSecurityTrustResourceUrl(CoreFile.convertFileSrc(url));
@ -181,7 +188,6 @@ export class CoreIframeComponent implements OnChanges, OnDestroy {
this.init();
});
}
}
/**
* Open help modal for iframes.

View File

@ -504,7 +504,7 @@ export class CoreUrlUtilsProvider {
*/
removeProtocolAndWWW(url: string): string {
// Remove protocol.
url = url.replace(/.*?:\/\//g, '');
url = url.replace(/^.*?:\/\//, '');
// Remove www.
url = url.replace(/^www./, '');