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

Feature: add support for serving from custom path #801

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

Conversation

Squixx
Copy link

@Squixx Squixx commented Apr 3, 2024

Description

I want to run dependency track frontend on a path instead of /, but doing this currently would require to do a full rebuild of the frontend. This PR could be a possible solution direction.

I've seen the reference to:

 // Relative paths cannot be supported. Research by @nscur0 - https://owasp.slack.com/archives/CTC03GX9S/p1608400149085400
  publicPath: '/',

However I can not open this link, so am oblivious to any previous research

I'll update the docs once it's clear this is the way forward

Addressed Issue

DependencyTrack/dependency-track#1514

Additional Details

setting the BASE_HREF env variable allows you to set a path

Checklist

@Squixx Squixx changed the title Feat/urlpath Feature: add suport for serving from custom path Apr 3, 2024
@Squixx Squixx changed the title Feature: add suport for serving from custom path Feature: add support for serving from custom path Apr 3, 2024
Martijn van der Meij added 4 commits April 3, 2024 16:47
Signed-off-by: Martijn van der Meij <martijn.vandermeij@vattenfall.nl>
Signed-off-by: Martijn van der Meij <martijn.vandermeij@vattenfall.nl>
Signed-off-by: Martijn van der Meij <martijn.vandermeij@vattenfall.nl>
Signed-off-by: Martijn van der Meij <martijn.vandermeij@vattenfall.nl>
@fnxpt
Copy link
Contributor

fnxpt commented Apr 23, 2024

similar to #23

@Squixx
Copy link
Author

Squixx commented Apr 23, 2024

similar to #23

Does that work with oidc? This pr also has working oidc, but I'd be happy if either could move forward

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants