Skip to content

fix(build): stripe dynamic import + API version pin removal + CSRF proxy headers + policy coverage + lint fixes#9

Merged
thefiredev-cloud merged 5 commits intomainfrom
chore/netlify-build-fixes
Oct 11, 2025
Merged

fix(build): stripe dynamic import + API version pin removal + CSRF proxy headers + policy coverage + lint fixes#9
thefiredev-cloud merged 5 commits intomainfrom
chore/netlify-build-fixes

Conversation

@thefiredev-cloud
Copy link
Copy Markdown
Owner

Summary

  • Fix Netlify build error by avoiding client bundling of server-only Stripe helper (dynamic import in payments service)
  • Remove future-dated Stripe API version pins; rely on SDK default to prevent runtime rejections
  • Add proxy-aware CSRF origin validation (x-forwarded-proto/host) to avoid false negatives behind Netlify/CDN
  • Align emails service ordering with tests (order by created_at)
  • Close authorization policy coverage gaps (admin-only placeholders for missing methods) to reach 100%
  • Fix secret scan grep option bug (use -e) to reduce false positives
  • Bump @netlify/plugin-nextjs to ^5.13.5 (per Netlify hint)

Files Changed (highlights)

  • lib/supabase/services/payments.ts
  • lib/supabase/services/StripeWebhookHandler.ts
  • lib/supabase/services/stripeCustomer.ts (added, required by payments)
  • lib/csrf.ts
  • lib/supabase/services/emails.ts
  • lib/supabase/middleware/authorization.ts
  • lib/utils/html-sanitization.ts
  • lib/validation/email-schemas.ts
  • scripts/check-secrets.sh
  • package.json
  • docs/100-percent-completion-plan.md

Notes

  • No behavior change for production payments; previews run in test-mode if ENABLE_PAYMENT_PROCESSING is false or Stripe config missing
  • MCP-related areas untouched

Deployment

  • This PR should trigger a Netlify deploy preview; build should pass with the prior 'module not found' error resolved.

…e-dated Stripe API pins; add proxy-aware CSRF origin check; align email ordering; complete auth policy coverage; fix secret-scan grep; bump Netlify plugin
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 11, 2025

Deploy Preview for bucolic-cat-5fce49 failed. Why did it fail? →

Name Link
🔨 Latest commit 79cc9f0
🔍 Latest deploy log https://app.netlify.com/projects/bucolic-cat-5fce49/deploys/68eaea4f0f420a0008bfb263

@supabase
Copy link
Copy Markdown

supabase Bot commented Oct 11, 2025

This pull request has been ignored for the connected project mdzzslzwaturlmyhnzzw because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@thefiredev-cloud thefiredev-cloud merged commit 9f76033 into main Oct 11, 2025
7 of 14 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