feat: add French localization strings

Merge pull request #1545 from ascillato/patch-2

Update Spanish Translation

Update Spanish Translation

Update Spanish Translation.

Added missing translations. Fixed some typos.
Also, some changes were made to use a more formal and international spanish.

Initiate connection from RFB constructor

We need to do this in order to safely attach to existing WebSocket
objects. There may be message events already pending so we must set up
our event handlers before returning.

This means we will now throw errors instead of generating "disconnect"
events on problems as the caller no longer has the opportunity to set up
event handlers.

This might have been the correct approach from the start as it mimics
how e.g. the WebSocket constructor works.

Don't fake open events in Websock

We don't know if the caller is prepared to receive those events right
now as normally they would get them on a fresh new stack later. We also
can't delay delivery since then we might deliver the event after any
pending "message" events.

Better to push the problem one layer up to the caller, which now needs
to be more aware of the state of the WebSocket object it is trying to
use.

Refuse to use already closed WebSocket objects

We can't do anything useful with them anyway.

Provide readyState for Websock objects

It mainly reports the state of the underlying object in consistent
manner.

Move Websock event handlers to own methods

Avoid cluttering up the RFB constructor.

Add unit tests for connect/attach errors

Add unit tests for passing WebSocket objects

Stop explicitly testing connection states

These are internal and we should be testing the externally observable
behaviour.

Revert "Fixed a race condition when attaching to an existing socket"

This reverts commit ef27628c6dff6120b0ed0d4728cc6e8a32b7be53. By
bypassing setTimeout() it creates other race conditions so this is not
the proper fix for the issue.

Merge pull request #1537 from TimSBSquare/bugfix/existing-channel-race-condition

Fixed a race condition when attaching to an existing socket

Fixed a race condition when attaching to an existing socket

This is an error that presents itself with RTCDataChannel's, I suspect this could not
happen with a pre-existing WebSocket.

If the remote connection creates a data channel then the local (VNC) side gets a channel
created callback. It may also be the case that in that very same tick the socket is also
opened and buffered data received. This meant that (in my tests) about 1/3 of the time
noVNC would fail to respond to the initial message from the server because it was received
and subsequently not handled during that initial tick.

Also made the documentation reflect this new behaviour and document the existing behaviour.

Add Portuguese (Brazil) translation

Add VeNCrypt Plain authentication tests

This change adds tests for the VeNCrypt Plain authentication. In doing
that, this also fixes a typo that was introduced in a recent change.

Merge branch 'homogenize-credentials-testing' of https://github.com/lhchavez/noVNC

Merge branch 'feature/support-existing-rtcdatachannel-or-websocket-squashed' of https://github.com/TimSBSquare/noVNC

Normalize the credentials presence check

Most places that check for the presence / absence of credentials compare
them against `undefined`, except the one for Plain authentication.

This change makes the very last place to use the same pattern (instead
of checking for falsiness) for consistency. Additionally, there are ways
to configure PAM to accept empty passwords, so it's possible for a user
to legitimately send an empty string as password.

Added support for RTCDataChannel

This work is originally by Ryan Castner <castner.rr@gmail.com> and
submitted as a PR here https://github.com/novnc/noVNC/pull/1362

Architecturally it is much the same except it doesn't rename a lot
of variables to make this more reviewable. It also avoids unrelated
changes such as replacing .onclose with an event listener, which
caused numerous test failures.

It also adds in ppoffice's fix to initialise the buffers.

Like the original author I don't have enough time available to
refactor this project to the new style event listeners.

Review cleanup for RTCDataChannel support (see below)

* More descriptive error when url or channel not set.
* Moved websocket property check to WebSock.
  This had unintended consequences in the tests that required some
  fixup. Mostly due to some tests not always passing FakeWebsocket.
  FakeWebsocket also needs to set the listeners to null to be compatible
  with what is in thw browser and expected by the property check code.
  The property check code now also takes into account class prototypes
  for test compatibility.
* Removed unreachable code.
* Reverted comment.
* Cleanup raw channel reference in rfb on websock close.
* Use readyState to check whether a socket is open rather than assuming.
* Updated RFB constructor documentation

Removed an unused boolean passed to attach

Allow longer passwords in Plain authentication

Some people have longer passwords than 256 characters (hooray for
password managers!). Server implementations also allow longer passwords:
TigerVNC allows up to 1024 characters.

Change phrasing for discussion group link

It's listed with the issue templates, so make sure it follows the same
style in phrasing.

Add link to discussion group from issues

So that people can easily find it and not file bug reports for
things that are just questions.

Hide link to create blank issues

We want users to use the templates so we don't miss any relevant
information.

Update Japanese translation

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

Update conversion documentation for Node.js

We now only support conversion to CommonJS, in order to support Node.js
older than version 15.

Remove documentation about converting the app

This is no longer possible as we now require browser support for
modules.

Use snap actions instead of the broken container

The container didn't work properly for our base snap anyway.

Package files directly in snapcraft.yaml

We don't need to convert things anymore, so reference files directly in
the snap yaml file.

Store result from NPM and snap builds

To ease debugging.

Run NPM and snap builds on every push

To make sure these things still build. That means we need to make the
actual deploy parts optional.

chore: sync with W3C documentation

https://www.w3.org/TR/uievents-key/

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

Remove some unnecessary use of done argument

Fake key releases for some Japanese IM keys

Windows behaves very oddly for some Japanese IM keys in that it won't
send a key release event when the key is released. In some keys it never
sends the event, and in some cases it sends the release as the key is
pressed the subsequent time.

Send combination keysyms for some Japanese keys

Windows doesn't give us stable symbols for a bunch of Japanese IM keys,
instead alternating between two symbols. This state is not synchronised
with the IM running on the remote server so to have stable behaviour we
have to collapse these multiple symbols in to a single keysym.

Fix typo for ZenkakuHankaku key

Use toggle keysym for Eisu key

This matches how the key behaves on a Linux system.

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

Remove createEvent() fallbacks

We can now rely on proper constructors for our events.

Remove createImageData() fallback

All our browsers should be new enough now that we can rely on the
ImageData constructor.

Remove many small, obsolete, old browser hacks

These are for browsers no longer supported anyway.

Use Fetch API for getting JSON data

We no longer need to support Internet Explorer so we can use a more
proper API here.

Remove unused injectParamIfMissing()

It should have been removed in 58fc267b2bac1b89450b76a21e91ffd25e7e4358
with the caller.

Update keycode mappings to latest version

This update fixes Korean layouts.

Merge branches 'logs' and 'timeout' of https://github.com/jcpunk/noVNC

Make timeouts/heartbeats easy to setup

Permit setting syslog on websockify

Remove keypress handling

We no longer support any browser that requires this legacy handling.

Drop support for legacy Edge

Remove legacy conversion of modules

We no longer support Internet Explorer so we can now require that
browsers support modules.

Some conversion to commonjs still remains for nodejs.

Drop support for Internet Explorer

Include current websockify in noVNC snap

The one from the system is too old to support current versions of noVNC,
so we need to bundle our own.

Add workaround for Firefox PNG rounding bug

Remove Firefox Alt workaround

The bug got fixed way back in Firefox 63, and it is also misbehaving
with modern Firefox as they no longer consider AltGr an Alt-key.

Handle empty rects in RAW decoder as well

It was overlooked in the previous commit because we couldn't feed
empty data messages through the test framework.

Ignore recording variables in our linter

Approximate comparison of JPEG result

The browsers have various rounding errors so we need to compare the
output data only approximately and not exactly.

Handle empty rects from the server

These are very pointless for the server to send, but not a violation of
the protocol so we need to be able to handle them. We've seen this
happen in real world scenarios a few times.

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

Fix call to _recvMessage()

Missed to rename this one in commit ea858bfa275d85541ac26d035967959714e8e255

Fix names of recording variables

The name of these variables must match how they were set when the
recording was created.

Reverts part of 95632e413d75030c49577c0e5cf1003208a5ba09

noVNC 1.2.0

Update generated JS files for translations

Update Swedish translation

Merge pull request #1435 from nnn1590/fix-japanese-translation

Fix Japanese translation

Explicitly set NPM registry URL

Otherwise the action doesn't configure npm properly to authenticate and
our publish actions will fail with 404 errors.

noVNC 1.2.0 beta

Remove note about clipboard ISO 8859-1 restriction

We now support full Unicode, provided the server also supports the
proper extension.

Update translation template file

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

Publish pre-releases to beta channels

Extra important for NPM which doesn't allow replacing a release once it
is published.

Mention gestures in README

Deploy new snap on release

Move snap specific script to snap directory

Remove redundant VERSION file

We want to avoid having this in multiple places.

Don't detach handler if it doesn't exist

If the beforeEach() step has been skipped then there won't be anything
here to detach.

Handle quick Cursor detach after mouse up

This timer might fire after the Cursor object has detached from a DOM
element, causing crashes. This will likely not happen in real scenarios,
but the tests are quick enough to trigger this.

Update build badges for GitHub actions

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

Use GitHub actions instead of Travis/Sauce Labs

The ability to use Sauce Labs for pull requests has now stopped working,
as Travis warned about several years ago. Instead run our tests directly
on GitHub on their various virtual machines.

Remove default Karma options

No need to mention things where we already use the default value.

Alway focus on touchstart

The new gesture detection code will always prevent the default behaviour
of touchstart, so this check no longer works properly. We might want to
add something similar to GestureHandler in the future, but let's wait
and see what use cases are requested.

Compensate for visual viewport when moving cursor

Fix Japanese translation

Merge pull request #1414 from CendioOssman/gesture

Add gesture handling

Fake cursor position when using touch

With the new gestures we will simulate the cursor being in a different
location than any of the touch points. This is a bit too complex for the
Cursor class, so let's just explicitly tell it where we want the cursor
rendered.

Move mouse event handling to RFB class

Move the last remaining bits to the RFB class to keep things simple, as
the Mouse class no longer provides any real value.

Increase wheel step threshold

The previous value made the detection too sensitive and it was very
difficult to scroll precisely. A value of 50 pixels should give similar
behaviour to systems that don't do fine grained scrolling.

Move wheel event handling to RFB class

The Mouse class does very little now so it mostly just obfuscate things.
Move everything directly in to the RFB class instead.

Remove very legacy mouse event handling

This is only needed on such ancient versions of Internet Explorer that
it wouldn't satisfy our other requirements anyway.

Remove delayed wheel timer

This isn't really expected behaviour from a user, i.e. that an extremely
small wheel movement still gives a large scroll event in the remote application.

Add lint rule for function declaration indentation

Add touch gestures for mouse emulation

Add several single and multitouch gestures to simulate various mouse
actions that would otherwise be impossible to perform.

This replaces the old system where you could select which mouse button
a single touch would generate.

Start fake test clock at real clock time

Some code relies on the clock having a somewhat sane value, so let's not
start at 0.

Avoid fractional pixel sizes from Display

Avoid printing the Websock buffer in tests

It takes forever and just messes up the output.

Switch to RGBx pixel format

This is what the browser wants so it avoids having to spend time
converting everything. Unfortunately it usually means the server instead
needs to convert it for us, but we assume it has more power than we do.

Standardise on a single blit function

Keep everything simpler by always blitting in the same pixel format.
It's up to the decoders to convert if they need to.