Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

📥 Pull Request

📘 Description
Fixes dashboard authentication and API communication issues in self-hosted AgentOps deployments by adding necessary environment variable propagation and client-side request proxying.

Changes:

  • app/compose.yaml: Pass NEXT_PUBLIC_API_URL to dashboard container at build time and runtime
  • app/dashboard/next.config.js: Add rewrites to proxy /auth/* and /opsboard/* requests from dashboard to API server

Problem Solved:
In self-hosted environments, the dashboard was unable to authenticate users or fetch data from the API because it couldn't properly route requests to the local API server.

🧪 Testing

  • ⚠️ Limited testing: Verified Docker Compose builds successfully and Next.js config syntax is valid
  • Requires human testing: Full authentication flow and dashboard functionality in self-hosted environment

🔍 Human Review Checklist

  • CRITICAL: Test complete login/authentication flow works in self-hosted setup
  • Verify dashboard can access /auth and /opsboard endpoints through rewrites
  • Confirm NEXT_PUBLIC_API_URL environment variable is properly set and accessible
  • Test that existing functionality isn't broken by the new rewrites
  • Validate in actual self-hosted Docker Compose environment

Link to Devin run: https://app.devin.ai/sessions/7c733bfe44374472882fafc78a2552b3
Requested by: @areibman

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@areibman areibman merged commit 5d9718e into main Sep 3, 2025
2 checks passed
@areibman areibman deleted the devin/1756861565-dashboard-proxy-clean branch September 3, 2025 01:08
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.

2 participants