mirror of
https://github.com/vercel-labs/portless.git
synced 2026-09-14 16:37:59 +08:00
fix/proxy-ipv6-loopback
5 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
|
974ae781cb |
chore: require pnpm release age policy (#307)
* chore: require pnpm release age policy - Add pnpm minimum release age 2880 and deny dependency build scripts. - Require Node 24 and pnpm 11 across engines, CI, and version files. - Refresh docs and lockfile for the runtime upgrade. * ci: install pnpm after node setup - Put Node 24 on PATH before pnpm/action-setup installs pnpm 11. - Keep setup-node cache restore after pnpm is installed. * ci: run javascript actions on node 24 - Opt CI and release workflows into GitHub JavaScript action execution on Node 24. - Unblock pnpm/action-setup from installing pnpm 11. * ci: avoid engine strict pnpm setup - Remove the root engine-strict npm config. - Drop the temporary Node 24 JavaScript action runtime override. * test: use valid ca pem fixture - Replace the fake CA text used in NODE_EXTRA_CA_CERTS tests with a valid PEM. - Avoid Windows Node 24 aborting before the capture script runs. |
||
|
|
a183e394a8 |
Config file support, zero-arg mode, and multi-app orchestration (#251)
* Add portless.json config file and zero-arg mode Adds an optional portless.json for configuring app name, script, and port without embedding portless in package.json scripts. Running bare `portless` from a workspace root auto-discovers pnpm packages and starts all dev scripts concurrently through the proxy. The --script flag overrides the default "dev" script for a single invocation. * Use npm scope as project base for multi-app names In multi-app mode, package names are now composed as subdomains of the npm scope (e.g., @og-sdk/docs becomes docs.og-sdk.localhost). Falls back to inferring the project name from the workspace root when packages are unscoped. * Pass through scripts that already invoke portless In multi-app mode, if a package's dev script starts with `portless`, run it as-is without registering a route or injecting PORT/HOST. The embedded portless invocation handles its own name and routing. * Fix multi-app PATH and deduplicate project name Augment PATH with node_modules/.bin from each package's directory so local binaries (tsup, next, vite, etc.) are found. Skip the project suffix when the package name already matches it (avoids names like portless.portless). * Fix proxy start prompting for trust on every launch Cache the CA fingerprint in a marker file (ca.trusted) after a successful trust so subsequent proxy starts skip the OS security check and avoid re-triggering the macOS authentication dialog. Also switch docs and example dev scripts to clean "next dev". * Skip proxy route for build-only tools, support portless config in package.json Non-server commands (tsup, tsc, esbuild, etc.) are auto-detected and run without a proxy route. Adds "proxy" config field for explicit control. Also supports a "portless" key in package.json — either as an object or a string shorthand for the app name. * Address PR review: config fixes, proxy auto-start, and refactors - Rewrite splitCommand to respect quoted arguments - Use frequency count for commonScope instead of first match - Extract ensureProxyRunning helper; call from multi-app mode - Remove loadConfig directory walk-up (check cwd only) - Deduplicate clearTrustMarker call in generateCA - Remove proxy field from config (system-level, not app-level) - Fix early script filter to respect per-app overrides - Add portless key to google-oauth example - Replace invalid JSON comments with markdown headings in docs - Extract spawnProxiedApp/spawnTaskApp helpers from handleDefaultMulti * Address PR review: docs fixes, config precedence, proxy option - Fix docs claiming config walks up directories (it does not) - Clean up resolveAppConfig to use explicit field return - Add JSDoc to isServerCommand explaining denylist design - Add comment on sequential proxy spawning (port collision) - Handle backslash escapes in splitCommand - Remove non-null assertion on discovered proxy state - Pass appConfig through to avoid redundant disk read - Fix config precedence: package.json portless key wins (closest) - Implement proxy: false config option for non-server scripts * Add package manager delegation, turborepo support, and fix TLS state discovery - Add detectPackageManager and resolveScriptCommand to delegate script execution to the correct package manager (pnpm/yarn/bun/npm) instead of parsing shell scripts directly. This enables reliable turborepo integration where `portless` is the dev script. - Fix discoverState inferring tls=false when the proxy is running on port 443 but state files in /tmp were cleaned by the OS. - Re-discover proxy state in multi-app mode when proxy is already running. - Add turborepo integration docs and tests. * Address PR 251 review: child lifecycle, shell safety, line buffering, proxy dedup, multi-PM workspaces, config errors - Log exit codes per child and print summary in multi-app mode - Spawn child processes directly without shell for PM-delegated commands - Buffer partial lines in prefixed output streams, flush on end - Extract resolveProxyDesiredState helper to deduplicate proxy config - Support npm/yarn/bun workspaces via package.json "workspaces" field - Replace process.exit in config validation with ConfigValidationError * Address PR review: SIGKILL fallback, YAML flow sequences, glob wildcards, stream fixes - Add 5s SIGKILL timeout after SIGTERM in multi-app cleanup to prevent hanging when children ignore SIGTERM - Handle YAML flow sequence syntax (packages: [a, b]) in pnpm workspace parser, with JSDoc documenting the supported YAML subset - Fall through from multi-app to single-app mode when no workspace packages have the target script, so workspace roots with their own portless.json still work - Refactor expandSingleGlob into recursive expandSegments to support mid-path wildcards (packages/*/src) and prefix wildcards (apps/team-*) - Use StringDecoder in prefixStream for correct multi-byte UTF-8 handling across chunk boundaries, and preserve empty lines from child output * Eliminate /tmp state dir, fix symlink LPE, and clean up multi-app output Move all proxy state from /tmp/portless to ~/.portless, fixing both the macOS /tmp cleanup problem (repeated trust prompts) and the symlink LPE vulnerability (GHSA-p67g-wf2j-g3h5). Add lstatSync symlink guard in fixOwnership as defense-in-depth. Remove the interactive "Start proxy?" prompt in favor of auto-starting. Rewrite multi-app startup output so URLs stand out clearly. Also addresses PR 251 review items: re-throw ConfigValidationError, fix splitCommand backslash handling in single quotes, warn on unknown config keys, and remove unused export. * Fix macOS CA cleanup when multiple certs share the same CN Loop delete-certificate calls to remove all matching "portless Local CA" entries from each keychain. The previous single call failed with "is ambiguous, matches more than one certificate" when duplicate CAs had accumulated from repeated trust prompts. * Auto-trust CA after proxy auto-start When the proxy is auto-started in single-app or multi-app mode and the CA is not yet trusted, run handleTrust which has proper sudo elevation logic. This prevents the scenario where a new user cancels the trust dialog during proxy start and is left with browser cert errors and no clear recovery path. * Fix prettier formatting in configuration docs * Use turbo for multi-app orchestration When turbo.json is present, portless delegates to `turbo run <script>` instead of spawning each app individually. Per-app PORT, HOST, and PORTLESS_URL are injected via a lightweight --require loader so turbo retains dependency ordering, output grouping, and task graph awareness. Falls back to direct spawning when turbo is absent or when the config sets `"turbo": false`. * Fix turbo loader double-escape, cleanup guards, and direct-spawn route cleanup - Remove redundant manual backslash escape in loaderSource; JSON.stringify already handles escaping for JS string literals - Quote loader path in buildNodeOptions when it contains spaces - Add cleanedUp guard to runWithTurbo and runWithDirectSpawn cleanup functions to prevent double execution on signal + explicit call - Strip trailing \r in prefixStream so Windows \r\n output renders cleanly - Track routes in runWithDirectSpawn and remove them in the signal handler, matching the pattern already used by runWithTurbo * Fix doc gaps: state dir, CA path, turbo field, pkg portless key, hostname convention - State directory docs still showed the old 3-tier table (port < 1024 -> /tmp/portless). All state now lives in ~/.portless; updated SKILL.md and docs configuration page. - CA path references in README and SKILL.md still mentioned /tmp/portless/ca.pem; simplified to ~/.portless/ca.pem. - Added the `turbo` config field to all three config tables. - Documented the package.json "portless" key as a standalone feature (was only shown in the Turborepo section). - Documented the multi-app hostname convention (<package>.<project>.localhost derived from common npm scope). * Make bare portless just work everywhere, monochrome output - Remove config gates: bare `portless` and `portless run` resolve the dev script from package.json without requiring portless.json - Extend PORTLESS=0 skip path to handle zero-arg mode - Monochrome CLI output: bold for headers/errors, dim for warnings/muted, no color codes - Multi-app display: show full package name (with scope), alphabetized, URLs dimmed - Update docs, help text, and tests to reflect zero-config defaults * Improve multi-app output and turbo integration - Show full scoped package names in startup list, alphabetized - Hide task-only packages from output (they still run) - Use root package.json dev script for turbo when available, preserving user's flags like --concurrency and --filter - Pass extra args after -- through to turbo * Restore red/yellow ANSI colors for errors and warnings |
||
|
|
fee2c47d17 |
Default to HTTPS on port 443, with auto-sudo and HTTP-to-HTTPS redirect (#172)
* portless Fixes #123 and #145 * https * redirect * fixes * fixes * fixes * fixes * fixes * fixes * tests * fix tests * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes * fixes |
||
|
|
07508d3763 | fix ci failure (#120) | ||
|
|
fca96b0d40 |
v0.7.1
* oauth skill * v0.7.1 |