Skip to content

Secure Firebase config and add error boundary#10

Open
DigitalBlueprint239 wants to merge 1 commit into
masterfrom
codex/fix-firebase-security-and-add-error-boundaries
Open

Secure Firebase config and add error boundary#10
DigitalBlueprint239 wants to merge 1 commit into
masterfrom
codex/fix-firebase-security-and-add-error-boundaries

Conversation

@DigitalBlueprint239

Copy link
Copy Markdown
Owner

Summary

  • secure firebase by moving config to env vars
  • add basic ErrorBoundary component
  • wrap SmartPlaybook, RouteEditor, and CanvasArea with the boundary
  • include sample .env.local.example

Testing

  • npm test (fails: react-scripts not found)

https://chatgpt.com/codex/tasks/task_e_68821ffc845c832d90634d16da820a75

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bugbot free trial expires on August 7, 2025
Learn more in the Cursor dashboard.

Comment thread src/services/firebase.ts
storageBucket: import.meta.env.VITE_FIREBASE_STORAGE_BUCKET,
messagingSenderId: import.meta.env.VITE_FIREBASE_MESSAGING_SENDER_ID,
appId: import.meta.env.VITE_FIREBASE_APP_ID,
measurementId: import.meta.env.VITE_FIREBASE_MEASUREMENT_ID,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Firebase Initialization Fails Without Environment Validation

Firebase initialization fails at runtime with unclear error messages if required VITE_FIREBASE_* environment variables are undefined. The Firebase configuration object uses these variables without validation.

Locations (1)

Fix in CursorFix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant