Skip to content

fix: implement environment-based API proxy for cross-domain authentic… - #373

Open
daveaugustus wants to merge 1 commit into
mainfrom
fix/authentication-cross-domain-cookies
Open

fix: implement environment-based API proxy for cross-domain authentic…#373
daveaugustus wants to merge 1 commit into
mainfrom
fix/authentication-cross-domain-cookies

Conversation

@daveaugustus

Copy link
Copy Markdown
Contributor

…ation

  • Add Next.js rewrites to proxy /api/v1/* and /api/v2/* requests
  • Read API URLs from environment variables (NEXT_PUBLIC_API_URL*)
  • Resolve cross-domain cookie authentication issues
  • Enable proper session management and Create button visibility
  • No hardcoded URLs for security compliance

📃 Why Merge This PR?

Briefly describe the changes, fixes, or new features introduced in this PR.

🛠️ Issue Fixed

List any issues this PR resolves (e.g., Fixes #123).

🔍 PR Type

  • 💡 Feature
  • 🐛 Bug Fix
  • 📃 Documentation
  • 🎨 UI Improvements
  • 💻 Code Refactor
  • ✅ Tests

…ation

- Add Next.js rewrites to proxy /api/v1/* and /api/v2/* requests
- Read API URLs from environment variables (NEXT_PUBLIC_API_URL*)
- Resolve cross-domain cookie authentication issues
- Enable proper session management and Create button visibility
- No hardcoded URLs for security compliance
@netlify

netlify Bot commented Aug 12, 2025

Copy link
Copy Markdown

Deploy Preview for incredible-eclair-ada155 ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/projects/incredible-eclair-ada155/deploys/689b9678881eb88f488be73a
😎 Deploy Preview https://deploy-preview-373--incredible-eclair-ada155.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@daveaugustus daveaugustus added the 🐛 bug Something isn't working. label Aug 12, 2025
@piyusharmap piyusharmap added the ⏸ On Hold Keeping this issue on hold. label Aug 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working. bump:patch ⏸ On Hold Keeping this issue on hold.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants