Skip to content
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.

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

ui/docs: clone component is undefined when running as a server #669

Closed
coopernetes opened this issue Jul 31, 2024 · 0 comments · Fixed by #735
Closed

ui/docs: clone component is undefined when running as a server #669

coopernetes opened this issue Jul 31, 2024 · 0 comments · Fixed by #735
Labels
bug Something isn't working dashboard Part of CISO/OSPO experience to access UI good first issue Good for newcomers hacktoberfest

Comments

@coopernetes
Copy link
Contributor

coopernetes commented Jul 31, 2024

Describe the bug
When Git Proxy is deployed on a server, the "Clone" component under /admin/repo on the UI has an undefined value for the host.
It seems to stem from an unset environment variable VITE_SERVER_URI.

Ideally, this component would use the Origin URL and not require additional config when deploying Git Proxy's UI.

To Reproduce
Steps to reproduce the behavior:

  1. Deploy Git Proxy behind a load balancer and running on a server.
  2. Navigate to http[s]://:8080/admin/repo
  3. Click on the "Code" component on any of the listed repos.

Expected behavior
The frontend uses the server origin as the hostname for the clone URL.

Screenshot:
image

@coopernetes coopernetes added bug Something isn't working dashboard Part of CISO/OSPO experience to access UI good first issue Good for newcomers labels Jul 31, 2024
JamieSlome added a commit that referenced this issue Oct 2, 2024
fix: #669-ui/docs: clone component is undefined when running as a server
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dashboard Part of CISO/OSPO experience to access UI good first issue Good for newcomers hacktoberfest
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants