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.

Sign up for GitHub

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

Jump to bottom

Prefix websocket path with server path #1745

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

Prefix websocket path with server path #1745

wants to merge 2 commits into from

Commits on Jan 14, 2023

  1. Prefix websocket path with server path 

    Fixes novnc#1737 and it should work for all of the different scenarios where the relative assets also work.
This includes, for example:
   - /
   - /vnc.html
   - /vnc/
   - /vnc/vnc.html

It won't work for:
 - /vnc (but the assets don't work either, so it's consistent)
 - or /vnc.html/ (but this results in a 404 by default anyway)
    @vsilvar
    vsilvar committed Jan 14, 2023
    Configuration menu
    Copy the full SHA
    b7ef484 View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2023

  1. Replaced deprecated substr method

    @vsilvar
    vsilvar committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    b0c8a2b View commit details
    Browse the repository at this point in the history