Merge pull request #1318 from chrisjdev/emptyPassword

Allow connecting with empty string credentials

Allow connecting with empty string credentials

Checking for undefined instead of falsy. That way an empty strings are allowed.

Add encodeUTF8 function to core/util/strings.js

Make Cursor.detach() safe to call when not attached

Avoids having checks in higher layers.

Fix chinese translation errors

Thanks to @QQ2017 for providing the translation strings

Simplify logic for status hierarchy

Removes unnecessary variable

Ensure warning status timeouts are honored

When showing a new status popup we want to set a timer for how long to
show it. In cases where we show many statuses in a fast succession we
need to remove any running timeouts when showing a new one.

There are exceptions when new statuses won't be shown, and thats if a
more severe status is already showing, i.e and error or a warning.

Warnings can still have timeouts. There was a bug that occured when we
tried to show a normal status while a warning was showing. The bug
caused the warning status timeout to be removed even if the normal
status was never shown. We should only remove running timeouts if we're
actually going to show a new status.

Merge branch 'babel' of https://github.com/CendioOssman/noVNC

Don't use arrow functions in legacy loader

The browsers that need the legacy code do not support such fancy
modern things.

Load support scripts first

E.g. SystemJS requires the Promise polyfill, so make sure all our
support files are loaded first.

Upgrade to latest babel

There has been a lot of renaming and restructuring in babel, so we need
to modify our code to handle the latest version. We also need to adjust
the way we build our babel worker as babel itself no longer runs in older
browsers such as Internet Explorer.

Merge pull request #1319 from wavezhang/patch-2

Update zh_CN.po

Always include Promise polyfill for legacy browsers

It is now used by our general code and not just by the conversion
routines, so we need to make sure it is always included for the
old browsers.

Clean up handling of untransformed files

This control flow is difficult enough as it is to follow. Move the
handling of the untransformed files to a separate block to make it
slightly easier to understand.

Remove redundant "no copy" check

We also check this list as a filter to walkDir(), so no need for the
extra check here.

Use proper backticks for generating legacy script tags

Update zh_CN.po

fix spell errors, improve translations

Fix missing caps lock events on iOS

Caps Lock on iOS only trigged key release or key press events.
When it's clicked it would only send keydown, and next time
it would only send keyup and so on. It should send both a key press
and a key release.

Also added the unit tests for macOS since those were missing.

Co-Authored-By: Alex Tanskanen <aleta@cendio.se>

Add README to app/locale warning not to modify

Add "macOS shuffle" to iOS as well

Since iOS functions like macOS with regards to Alt behaving like AltGr,
we need the same workaround on iOS as well.

Handle missing Shift events on Windows

This is a bug in the OS that leaks through to the browsers. We need
to fake a Shift release here to avoid Shift getting stuck in the remote
session.

Send NumLock on macOS, even though the key is Clear

There is no obvious choice what works best here, but this is what
TigerVNC has been doing for years without complaints. Let's follow
them until we get reports that this doesn't work well.

Fix typo for MailSend key

Update to latest UI Events key specification

Fix AltGr for a few more keys in IE and Edge

Some keys apparently send 'Unidentified' rather than an unshifted value.
Make sure those are also handled. Examples are \ and | on a Swedish
keyboard.

Keep the virtual keyboard after using extra keys

If using the extra keys always gives focus to the screen then an
on-screen keyboard would be closed. When using on-screen keyboards we
instead want to give focus to our virtual keyboard input element.

Revert iOS keyup workaround

It seems Apple has fixed their buggy keyup events, so remove the
workaround and allow keys to be kept pressed again.

This is a revert of 9e99ce126ca8f6f350fa015c0e58d35103c62f7e.

Handle broken numpad delete key in Chrome

Explain why Clear maps to KP_Begin

Try to handle Meta key properly

The standards have unfortunatly caused some confusion between the Windows
key and the original Meta key. Try to handle the common case sanely at least.

Merge pull request #1312 from samhed/master

Fix so that you can use the keyboard after using the extra keys

Move focus to the screen when using extra keys

A regression from 2afda54 and friends was that you couldn't use the
extra keys and then directly use the keyboard, you would have to click
in the session first.

This commit restores the correct behavior and also adds a visual queue
to the fact that the screen got the focus by fading the controlbar.

Add missing scancode for sendTab

Rename document.capturedElem to captureElement

To better fit most naming.

Improve Windows key image

The previous one didn't have sharp lines, or follow pixel boundaries
properly.

Merge pull request #1309 from samhed/disappearing_cursor

Fix disappearing cursor

Hide the emulated cursor when target is null

Makes it easier to understand what happens when a real element isn't
passed as a target to updateVisibility(). Also makes the code more
robust to future changes.

Co-authored-by: Alex Tanskanen <aleta@cendio.se>
Co-authored-by: Niko Lehto <nikle@cendio.se>

Fix disappearing cursor after click

In the cursor emulation when deciding if the cursor should be hidden -
Instead of checking what's under the cursor, we check the element that
has capture.

This introduced another bug in the cursor emulation. The cursor did not
always disappear properly when using our cursor emulation together with
our setCapture polyfill. More specifically, we saw a problem when a
capture ended on an element without cursor emulation.

We solved this by introducing another visibility check on a timer in
the cursor emulation. However this led to yet another problem where
this timer conflicted with the timer in the setCapture polyfill.

We removed the timeout in the setCapture polyfill and created a
variable to make sure that all the events remaining in the queue can be
completed.

Co-authored-by: Alex Tanskanen <aleta@cendio.se>
Co-authored-by: Niko Lehto <nikle@cendio.se>

Check next elem at mouseleave in cursor emulation

It's not obvious that we want to hide the cursor when we get a leave,
it depends on the element that we're leaving to. This makes the code
more robust.

Co-authored-by: Alex Tanskanen <aleta@cendio.se>
Co-authored-by: Niko Lehto <nikle@cendio.se>

Rename variables in setCapture proxy

The names of many variables were too similar. To make the code easier
to follow we renamed:

* _captureElem to _capturedElem
* _captureElemChanged() to _capturedElemChanged()
* captureElem to proxyElem
* elem to target

Co-authored-by: Alex Tanskanen <aleta@cendio.se>
Co-authored-by: Niko Lehto <nikle@cendio.se>

Fix alt text for drag button

We had left an old placeholder text on this button.

Never show drag icon if clipping is disabled

Toggling the enabled state is a remnant from an earlier version
of the code where we could determine if the the session is actually
clipped, and not just that the setting is enabled.

Right now we only change things based on the setting, so let's
completely hide the button when clipping is disabled.

Update copyright to 2019 for modified files

Code comments for how the receieve queue works

Clarify why we ENABLE_COPYWITHIN is false

Merge pull request #1298 from CendioNiko/edgeCursorURI

Fix url cursor detection on Edge

Merge pull request #1299 from CendioNiko/vmwarecursor

Add support for VMware cursor encoding

Merge pull request #1289 from CendioNiko/master

Desktop name improvements

Merge pull request #1281 from jalfd/optimize-receive-buffer

Optimize receive buffer

Clarify comments for broken alt in FF on Windows

Merge pull request #1280 from jalfd/alt-key-recursion

Avoid recursion in Alt check on Firefox

Merge pull request #1279 from CendioOssman/nomodule

Use "nomodule" instead of manual check

Use "nomodule" instead of manual check

Very few browsers are left in the wild that supports modules but not
"nomodule", so let's simplify our handling a bit.

Safari 10 supports modules but not 'nomodule', this means that this
particular version of Safari will be broken. Due to this we have to
bump up the required Safari version to 11.

Add support for VMware cursor encoding

Supports both classic cursor type and alpha cursor type. In classic
mode the server can send 'inverted' pixels for the cursor, our code
does not support this but handles these pixels as opaque black.

Co-authored-by: Samuel Mannehed <samuel@cendio.se>

Fix url cursor detection on Edge

_supportCursorURIs was set to true even when Edge didn't support
URIs because the fallback value "default" was used.

Restore page name after disconnect

Test unicode desktop names

Add support for DesktopName extension

This extension allows session name to be changed during runtime.

Set viewport size for autoscale tests

We were incorrectly relying on the viewport being indirectly set
for us. Make sure we are explicit in what we want for these tests.

Merge branch 'upgrade' of https://github.com/CendioOssman/noVNC

Remove pointless Display.clear()

It served no meaningful purpose and it had bugs. So let's remove it
rather than try to fix it.

Remove unused Display.logo attribute

Deprecate showDotCursor option for RFB constructor

It is not relevant for the connection stage so it should not have
been a constructor argument to begin with. Ship with a warning for
a release before we remove it.

Make sure showDotCursor can be modified before connecting

The cursor object is only attached to our canvas whilst connecting,
so we need to make sure we don't try to update anything when were
not connected or we'll get a crash.

Regenerate module loader after tool upgrades

Use latest versions of development dependencies

Let's make sure we get the latest features and fixes for all the
tools we are using.

Use latest NodeJS version in Travis

We want the latest and not be stuck on some old version. Otherwise
our scripts might not execute correctly.

Make sure translation tools are lint checked

They do not have a .js suffix so eslint isn't finding them
automatically.

When compacting the receive buffer don't copy unused buffer space

When compacting the receive buffer, we should only copy the bytes
between _rQi and _rQlen (the index of the first unread byte, and the
next write position).

Previously, we copied everything for _rQi up untill the end of the
buffer.

Don't compact the receive buffer unless we've actually run out of space

Previously, we would compact the buffer (moving unread data to the
start of the buffer) as follows:

- after processing a message, if there are zero unread bytes, just reset
  the indices for first and last unread byte to zero
- else, if at least 1/8th of the buffer is used, copy remaining data to the beginning of the buffer

The second option is never actually necessary, as before inserting new data
into the array, we already check if there's enough free space, and
compact the buffer first if necessary. So we've been doing a lot of
copies that weren't actually needed. Let's not do that any more.

Avoid recursion in Alt check on Firefox

The Firefox workaround which checks for missing Alt key events may
synthesise new KeyboardEvents. On these events, checkAlt should not be
recursively triggered. Otherwise, we get "too much recursion" errors
whenever the Alt key is pressed.

Fix trivial lint issues in translation tools

Indentation, missing semicolon, etc.

Upgrade to latest rollup

Remove unused import from module loader

Upgrade to latest sinon and chai

Avoid deprecated called.once from sinon-chai

It's been removed in newer versions and will break eventually.

Added a Snap package for noVNC (#1231)

Creating an Ubuntu Snap package to make noVNC easier to deploy.

Checks for the websockify binary in both the PATH (using which) and in the location where the Snap package places the binary. This is necessary for noVNC to be usable in a Snap. It doesn't affect the original functionality of git cloning websockify if it's not found in PATH or the Snap location.

Add missing parentheses for arrow func arg

Our lint tests expect this when an arrow functino has a body with
curly braces.

Add documentation for the option showDotCursor

This is not only a property, it's also a parameter to the constructor.

Remove the default value of wsProtocols

Using the 'binary' protocol by default is very non-standard.

Merge pull request #1262 from shiramax/sub_protocols

Add support in websocket sub-protocols

Add support in websocket sub-protocols

Merge pull request #1265 from juanjoDiaz/add_version_number_to_UI_2

Add version number to UI

Add version number to UI

Update link to websock.js API

Merge pull request #1259 from lyarwood/launch.sh

launch.sh: Check for a local websockify directory

launch.sh: Check for a local websockify directory

Previously launch.sh would check both for the existence of a local
websockify file and /websockify/run file.

This initial check should really be for a local websockify directory
as in packaged environments a file could very well be the actual
executable leading to launch.sh incorrectly attempting to use a local
version of websockify.

Merge branch 'add-japanese-translation' of https://github.com/nnn1590/noVNC

Add support for separate key file in launch script

Add Japanese translation

Revert "Fullscreen from iframe (#1236)" (#1247)

This reverts commit 19cdc15aa314760446866a1bcc2db99a80479683.

Remove server pixel format warnings

These are harmless and really only for debugging. So remove them
as they tend to trick people in to thinking something is wrong.
We already print the entire server pixel format earlier anyway in
case we need the details.

Fullscreen from iframe (#1236)

* First attempt to make the fullscreen button work inside an iframe.

* Cleaner distinction between document element and document.

* Scoping corrections. Auto-detect correct iframe.

* Added comments to the relevant sections.

* IE issue fixed.

* Same source issue solved. fullscreenToggle now checks if it is permitted to inspect other iframes.

Don't do cleanup on deploy

Travis wipes out node_modules otherwise, which we need for the prepare
stages of the deployment to npm.

noVNC 1.1.0

Update generated JS files for translations

Update Dutch translations

Co-authored-by: Arend Lapere <arend.lapere@gmail.com>

Update Swedish translations

Allow autoscale() with zero height or width

Commit 6e7e6f9 stopped the function from running if width or height was
zero, this commit reverts that change. This commit also makes the
resulting canvas 0x0 if autoscale is called with zero. By adding this
special case we can avoid division by zero in the calculations.

Get rid of self-closing tags

This is a remnant of XHTML and not used in standard HTML. Get rid
of them so that no-one mistakes our files for being XHTML compatible.