Skip to content

Conversation

@julianiff
Copy link
Contributor

@julianiff julianiff commented Sep 5, 2025

-> will use https as default when running dev server for apps.

@julianiff julianiff requested a review from SamuelAlev September 5, 2025 13:22
@julianiff julianiff requested a review from a team as a code owner September 5, 2025 13:22
@changeset-bot
Copy link

changeset-bot bot commented Sep 5, 2025

🦋 Changeset detected

Latest commit: 96a023c

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

This PR includes changesets to release 1 package
Name Type
@frontify/frontify-cli 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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 5, 2025

"prompts": "^2.4.2",
"vite": "^5.4.19",
"vite-plugin-externals": "^0.6.2",
"vite-plugin-mkcert": "^1.17.8",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is generating certificates out of our controls, I would go with self-signed instead.

Here is a list of security issues which happened with mkcert: https://security.snyk.io/package/linux/chainguard%3Alatest/mkcert

Copy link
Member

@SamuelAlev SamuelAlev Sep 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self signed on the fly (not bundled, it would be another security risk to add it in the CLI bundle) and cached in the user dir so it can be shared with multiple apps (and later blocks/themes) repositories.

@julianiff julianiff closed this Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants