Skip to content

fix(frontend): use relative API base and remove duplicated /api/v1 en… #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Life-00 merged 1 commit into from
Jan 19, 2026
Merged

fix(frontend): use relative API base and remove duplicated /api/v1 en… #18

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion frontend/src/config/config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
const config = {
// API Configuration
api: {
baseUrl: import.meta.env.VITE_API_BASE_URL || "http://127.0.0.1:8001",
baseUrl: import.meta.env.VITE_API_BASE_URL || "",

Copilot AI Jan 19, 2026

Copy link

Choose a reason for hiding this comment

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

This change updates the default baseUrl from an absolute URL to a relative path (empty string), which is appropriate for production deployment where the frontend and backend are served from the same origin via ingress routing. However, this PR is titled "remove duplicated /api/v1 endpoints" but only changes the config file.

There are hardcoded "/api/v1/..." paths in reportAgentService.js (lines 89, 114, 138, 159) that will now result in duplicate path segments like "/api/v1/api/v1/agents/report/analyze" when combined with the apiUrl getter. These should use the already-defined config.api.endpoints.agents.report paths instead (analyze, history, delete, download).

If these reportAgentService.js changes are in a separate commit or file not shown here, please ensure they are included. Otherwise, this PR is incomplete and will break the report agent API calls.

Copilot uses AI. Check for mistakes.
version: import.meta.env.VITE_API_VERSION || "v1",

get apiUrl() {
Expand Down
Loading