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

Context path change error #10958

Open
MuhannadBaraghith opened this issue Aug 27, 2024 · 0 comments
Open

Context path change error #10958

MuhannadBaraghith opened this issue Aug 27, 2024 · 0 comments

Comments

@MuhannadBaraghith
Copy link

MuhannadBaraghith commented Aug 27, 2024

I am using cBioPortal docker compose v6.0.12. I changed the context path for the cBioPortal by adding this argument: --path "/cbioportal" to the command:

/bin/sh -c "rm -rf /cbioportal-webapp/lib/servlet-api-2.5.jar && java -Xms2g -Xmx4g -cp '/cbioportal-webapp:/cbioportal-webapp/lib/*' org.cbioportal.PortalApplication --spring.config.location=cbioportal-webapp/application.properties --authenticate=false --path /cbioportal --session.service.url=http://cbioportal-session:5000/api/sessions/my_portal/"

as mentioned in this issue: https://github.com/cBioPortal/cbioportal/pull/9468/files/da63dbd1aad5b20830008295d9356925d7045cab

I also added: server.servlet.context-path="/cbioportal" to the application.properties.

When I query a study that I uploaded I get an error:

2024-08-27 10:16:19 2024-08-27T08:16:19.358Z DEBUG 15 --- [nio-8080-exec-1] o.s.web.servlet.DispatcherServlet : Failed to complete request: java.net.URISyntaxException: Illegal character in path at index 44: https://public.api.oncokb.org/api/v1��⢚+��K�][���[��\��[�S\�

I think it results from the file: ProxyController.java in line 99 and 100 where it tries to decode the path but does not remove the context path.

    String encodedPath = request.getRequestURI().replaceFirst("/proxy/A8F74CD7851BDEE8DCD2E86AB4E2A711/", "");
    String decodedPath = this.monkifier.decodeBase64(encodedPath);

Screenshot 2024-08-27 130516

Could you please fix that error? Thanks in advance for your help :)

I also could not find the documentation to change the context path, was it removed or updated?

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

No branches or pull requests

1 participant