studio/flamenco
studio/flamenco
12
30
Fork 38
Code Issues 64 Pull Requests 5 Releases Wiki Activity

Improvements to URL handling in the webapp allowing for SSL (when behind a proxy) #104296

Merged
Sybren A. Stüvel merged 5 commits from william-at-counter/flamenco:main into main 2024-04-11 15:00:55 +02:00
Conversation 5 Commits 5 Files Changed 2 +8 -6
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 042ab2be00 - Show all commits

2
web/app/package.json
View File

@ -12,7 +12,7 @@
}
],
"scripts": {
"dev": "vite --port 8081 --base /app/",
"dev": "vite --port 8081 --base /app/ --mode development",
"build": "vite build",
"preview": "vite preview --port 5050",
"lint": "eslint . --ext .vue,.js,.jsx,.cjs,.mjs --fix --ignore-path .gitignore"

12
web/app/src/urls.js
View File

@ -1,9 +1,11 @@
let url = new URL(window.location.href);
// Uncomment this when the web interface is running on a different port than the
// API, for example when using the Vite devserver. Set the API port here.
// if (url.port == '8081') {
// url.port = '8080';
// }
// When using Vite development mode, access the API on port 8080.
william-at-counter marked this conversation as resolved Outdated
Sybren A. Stüvel commented 2024-04-08 10:43:15 +02:00
Outdated
Review
Copy Link

The new comment doesn't explain why this piece of code is there to begin with. Please do keep the explanation that when using Vite in dev mode, the web app runs on a different port than the API.

The new comment doesn't explain why this piece of code is there to begin with. Please do keep the explanation that when using Vite in dev mode, the web app runs on a different port than the API.
if (import.meta.env.MODE == 'development') {
url.port = '8080';
}
console.log(import.meta.env.MODE);
const URLs = {
api: `${url.protocol}//${url.hostname}:${url.port}/`,