Skip to content

Fix undefined $node_version on hosts without Node, plus cleanup - #1

Merged
xpersonas merged 1 commit into
mainfrom
maintenance/lambda-node-fix-and-cleanup
Sep 30, 2026
Merged

xpersonas merged 1 commit into
mainfrom
maintenance/lambda-node-fix-and-cleanup

Conversation

@xpersonas

Copy link
Copy Markdown
Member
  • Default node_version to null so the endpoint no longer 500s when Node isn't installed (e.g. AWS Lambda)
  • Compare API keys with hash_equals(); return 401 for a missing request key and 500 only when the key isn't configured
  • Rate limit the endpoint to 60 requests per minute
  • Report the installed package version in php artisan about
  • Remove the unregistered StackReporter facade, unused guzzle dependency and unused InstalledVersions constructor argument
  • Move laravel/framework to require; bump testbench to ^9.0|^10.0
  • Rewrite the stale test suite and update CI to PHP 8.2-8.4 x Laravel 11/12
  • Update README to match the code; backfill CHANGELOG; add .gitignore

ClickUp Task

Technical Description

- Default node_version to null so the endpoint no longer 500s when Node
  isn't installed (e.g. AWS Lambda)
- Compare API keys with hash_equals(); return 401 for a missing request
  key and 500 only when the key isn't configured
- Rate limit the endpoint to 60 requests per minute
- Report the installed package version in `php artisan about`
- Remove the unregistered StackReporter facade, unused guzzle dependency
  and unused InstalledVersions constructor argument
- Move laravel/framework to require; bump testbench to ^9.0|^10.0
- Rewrite the stale test suite and update CI to PHP 8.2-8.4 x Laravel 11/12
- Update README to match the code; backfill CHANGELOG; add .gitignore

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@xpersonas
xpersonas merged commit b7adcf1 into main Sep 30, 2026
12 checks passed
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.

1 participant