Skip to content

Conversation

@ScriptedAlchemy
Copy link
Member

@ScriptedAlchemy ScriptedAlchemy commented May 7, 2024

Description

Allow manifest to be emitted with public path auto

adjust runtime plugin to look up public path off its own remote when setting remoteSnapshot into

Related Issue

#2449

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation.

@changeset-bot
Copy link

changeset-bot bot commented May 7, 2024

🦋 Changeset detected

Latest commit: 94b85a7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages
Name Type
@module-federation/dts-plugin Patch
@module-federation/managers Patch
@module-federation/manifest Patch
@module-federation/sdk Patch
@module-federation/enhanced Patch
@module-federation/rspack Patch
@module-federation/devtools Patch
@module-federation/nextjs-mf Patch
@module-federation/node Patch
@module-federation/runtime Patch
@module-federation/utilities Patch
@module-federation/webpack-bundler-runtime Patch
3008-runtime-remote Patch
@module-federation/modernjs Patch
@module-federation/runtime-tools Patch
@module-federation/third-party-dts-extractor Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented May 7, 2024

Deploy Preview for module-federation-docs ready!

Name Link
🔨 Latest commit 94b85a7
🔍 Latest deploy log https://app.netlify.com/sites/module-federation-docs/deploys/663d5ac863f95700089bcf96
😎 Deploy Preview https://deploy-preview-2463--module-federation-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ScriptedAlchemy ScriptedAlchemy changed the title feat(manifest): auto public path support for json remote feat(sdk): auto public path support for json remote May 7, 2024
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/manifest-auto-path branch from 290ddb1 to 4cdace4 Compare May 7, 2024 22:27
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/manifest-auto-path branch from 4cdace4 to edeae14 Compare May 7, 2024 22:28
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/manifest-auto-path branch from ede9b15 to b56a3d3 Compare May 8, 2024 20:50
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/manifest-auto-path branch from b56a3d3 to 67adcf1 Compare May 8, 2024 22:28
Comment on lines +52 to +54
return url
.replace(/#.*$/, '')
.replace(/\?.*$/, '')

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data

This [regular expression](1) that depends on [library input](2) may run slow on strings starting with '?' and with many repetitions of '?'. This [regular expression](1) that depends on [library input](3) may run slow on strings starting with '?' and with many repetitions of '?'.
Comment on lines +52 to +53
return url
.replace(/#.*$/, '')

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data

This [regular expression](1) that depends on [library input](2) may run slow on strings starting with '#' and with many repetitions of '#'. This [regular expression](1) that depends on [library input](3) may run slow on strings starting with '#' and with many repetitions of '#'.
@ScriptedAlchemy ScriptedAlchemy marked this pull request as ready for review May 9, 2024 23:40
@ScriptedAlchemy ScriptedAlchemy merged commit 2e52e51 into main May 10, 2024
@ScriptedAlchemy ScriptedAlchemy deleted the feat/manifest-auto-path branch May 10, 2024 04:43
@2heal1 2heal1 mentioned this pull request May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants