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

[Bug] Websocket not working with Cryostat exposed on a specific Path #1860

Open
ElfoLiNk opened this issue Jan 22, 2024 · 2 comments
Open

[Bug] Websocket not working with Cryostat exposed on a specific Path #1860

ElfoLiNk opened this issue Jan 22, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@ElfoLiNk
Copy link

Current Behavior

I'm trying to run Crystat under a path prefix inside kubernetes like hostname.com/cryostat.

To make the javascript load I had to use the 2.4.1-SNAPSHOT version but the websocket it's trying to call the API under the root /

Expected Behavior

Websocket should use correct endpoint when cryostat is exposed via an ingress under a subpath other than root /

Steps To Reproduce

No response

Environment

- Environment: AKS 1.27
- Version: 2.4.1-snapshot

Anything else?

No response

@ElfoLiNk ElfoLiNk added bug Something isn't working needs-triage Needs thorough attention from code reviewers labels Jan 22, 2024
@andrewazores
Copy link
Member

andrewazores commented Jan 22, 2024

I think this is a duplicate of https://github.com/cryostatio/cryostat/issues/1810 , though that bug "should" be fixed in the latest 2.4.1-snapshot images...

Sounds like the relative path is working properly for the JavaScript loading and perhaps API calls, but not for the Websocket path.

@andrewazores andrewazores removed the needs-triage Needs thorough attention from code reviewers label Jan 22, 2024
@andrewazores andrewazores self-assigned this Jan 22, 2024
@andrewazores andrewazores moved this to Todo in 2.4.1 release Jan 22, 2024
@andrewazores
Copy link
Member

The more I'm digging into this the more I'm finding bits and pieces that would need patching to take into account that the API paths may not be mounted on the root path. I'm not sure I will be able to prioritize work on this in the near term since we are only planning a 2.4.1 bugfix release from this codebase, and after that will be 3.0 which is a substantial rewrite and rearchitecture under the covers. From some preliminary testing I think what you're looking to achieve is already possible with 3.0, but there are a lot of other features missing so far to get that up to speed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants