Skip to content

Add configurable PUBLIC_URL for docker images #827

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

Open
Nani0002 wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add configurable PUBLIC_URL for docker images #827

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
bd8b535
Prepare server to accept url prefix
Nani0002 Nov 11, 2025
64244a5
Base path dynamic change
vpisti7 Dec 8, 2025
a0fad2e
Update docker README to include -e option for the webgui
Nani0002 Dec 15, 2025
7297c15
Update BASE_PATH and replace EoL characters
Nani0002 Dec 15, 2025
15ffd10
Fix CodeQL string escape issue
Nani0002 Dec 18, 2025
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
5 changes: 5 additions & 0 deletions docker/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,11 @@ docker run \
CodeCompass_webserver -w /workspace
```

If you want to edit the public URL of the web GUI, you can do this by appending the
`-e APP_BASE_PATH=/YOUR_PATH` command option to the previous run command.
Leaving this out of the command will start the web interface with no set path.
The new GUI will still be accessible in the /new path appended to the end of your URL.
Comment on lines +167 to +168
Copy link
Collaborator

@mcserep mcserep Jan 30, 2026

Choose a reason for hiding this comment

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

Without running patch-basepath.js, how will the new GUI be accessible on the given path? The documentation says nothing about executing it, it is only done in the Dockerfile.


# Official DockerHub images

Prebuilt images can be downloaded from DockerHub, from the
Expand Down
3 changes: 2 additions & 1 deletion docker/web/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,4 +76,5 @@ EXPOSE 8080

ENTRYPOINT ["tini", "--", "/usr/local/bin/entrypoint.sh"]

CMD ["CodeCompass_webserver", "-w", "/workspace", "-p", "8080"]
CMD ["sh", "-c", "node ../../webgui-new/scripts/patch-basepath.js && CodeCompass_webserver -w /workspace -p 8080"]
Copy link
Collaborator

@mcserep mcserep Jan 30, 2026

Choose a reason for hiding this comment

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

No, the source folder should not be in the Docker image or in the install location at all. There is an open PR #811 for that, but merging it required to fix this issue first. Instead, any script should be installed through CMake to the install location.


Empty file modified plugins/search/lib/java/lucene-memory-4.9.0.jar
100755 → 100644
View file
Open in desktop
Copy link
Collaborator

@mcserep mcserep Jan 30, 2026

Choose a reason for hiding this comment

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

File should be kept executable.

Empty file.
Empty file modified scripts/CodeCompass_logger
100755 → 100644
View file
Open in desktop
Copy link
Collaborator

@mcserep mcserep Jan 30, 2026

Choose a reason for hiding this comment

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

File should be kept executable.

Empty file.
Empty file modified scripts/install_latest_build2.sh
100755 → 100644
View file
Open in desktop
Copy link
Collaborator

@mcserep mcserep Jan 30, 2026

Choose a reason for hiding this comment

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

File should be kept executable.

Empty file.
Empty file modified scripts/remover.sh
100755 → 100644
View file
Open in desktop
Copy link
Collaborator

@mcserep mcserep Jan 30, 2026

Choose a reason for hiding this comment

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

File should be kept executable.

Empty file.
6 changes: 4 additions & 2 deletions webgui-new/next.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

require('dotenv').config();

const BASE_PLACEHOLDER = '/__URL_BASE_PATH__';

/** @type {import('next').NextConfig} */
const nextConfig = {
output: 'export',
Expand All @@ -13,8 +15,8 @@ const nextConfig = {
},
trailingSlash: true,
...(process.env.NODE_ENV === 'production' && {
assetPrefix: `${process.env.PUBLIC_URL || ''}/new`,
basePath: `${process.env.PUBLIC_URL || ''}/new`,
assetPrefix: `${process.env.PUBLIC_URL || ''}${BASE_PLACEHOLDER}`,
basePath: `${process.env.PUBLIC_URL || ''}${BASE_PLACEHOLDER}`,
}),
};

Expand Down
Loading