Skip to content

Add configurable PUBLIC_URL for docker images - #827

Open
Nani0002 wants to merge 5 commits into
Ericsson:masterfrom
Nani0002:612-Public_URL_with_docker_images
Open

Add configurable PUBLIC_URL for docker images#827
Nani0002 wants to merge 5 commits into
Ericsson:masterfrom
Nani0002:612-Public_URL_with_docker_images

Conversation

@Nani0002

Copy link
Copy Markdown

No description provided.

Comment threadwebgui-new/scripts/patch-basepath.js Fixed
@mcserepmcserep linked an issue Dec 18, 2025 that may be closed by this pull request
@mcserepmcserep changed the title 612 public url with docker imagesAdd configurable PUBLIC_URL for docker imagesDec 18, 2025
@mcserep
mcserep self-requested a review December 18, 2025 08:06
@mcserepmcserep added this to Lab and RoadmapDec 18, 2025
@github-project-automationgithub-project-automationBot moved this to In progress in LabDec 18, 2025
@github-project-automationgithub-project-automationBot moved this to In progress in RoadmapDec 18, 2025
@mcserepmcserep added this to the Upcoming Release milestone Dec 18, 2025
@mcserep

mcserep commented Dec 18, 2025

Copy link
Copy Markdown
Collaborator

@Nani0002 For some reason, all modified files in this PR show a complete change for them, not only for the affected lines. This makes hard to find the real changes for review; and will also mess up the history.

I guess you have changed the line endings and that is the reason we see that the entire files were changed. Please fix it. You shall commit all files with Linux file endings. (That's the default behaviour for Git even on Windows, unless configured otherwise.)

https://docs.github.com/en/get-started/git-basics/configuring-git-to-handle-line-endings

@mcserep

Copy link
Copy Markdown
Collaborator

I guess you have changed the line endings and that is the reason we see that the entire files were changed. Please fix it. You shall commit all files with Linux file endings. (That's the default behaviour for Git even on Windows, unless configured otherwise.)

https://docs.github.com/en/get-started/git-basics/configuring-git-to-handle-line-endings

For this reason, generating the TypeScript client from the Thrift specification fails as well:

Generating TypeScript files from Thrift...
thrift-codegen.sh: line 2: $'\r': command not found
thrift-codegen.sh: line 4: syntax error near unexpected token `$'in\r''
thrift-codegen.sh: line 4: ` case "$1" in

CI log:
https://github.com/Ericsson/CodeCompass/actions/runs/20231847990/job/58076534945

});

app.use('/', proxyHandler);
app.use("/", proxyHandler);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not needed change.

Comment threadscripts/remover.sh

Copy link
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Collaborator

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.

Comment threaddocker/web/Dockerfile
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
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Collaborator

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.

@mcserepmcserep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Multiple occurrences of the /new were not updated at all, e.g.:

Comment threaddocker/README.md
Comment on lines +167 to +168
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.

Copy link
Copy Markdown
Collaborator

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.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kind: Bug ⚠️Kind: Refactor 🔃Target: WebGUIIssues related to the web frontend.

Projects

Status: In progress
Status: In progress

Development

Successfully merging this pull request may close these issues.

Configurable PUBLIC_URL with docker images

4 participants

@Nani0002@mcserep@github-advanced-security@vpisti7