Bug 8495 - Clipboard incorrectly handles multiple CR+LF
Summary: Clipboard incorrectly handles multiple CR+LF
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Web Access (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.19.0
Assignee: Bugzilla mail exporter
URL:
Keywords: relnotes
Depends on: 8484
Blocks:
  Show dependency tree / graph
 
Reported: 2025-01-17 12:43 CET by Frida Flodin
Modified: 2025-01-22 10:54 CET (History)
1 user (show)

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Frida Flodin cendio 2025-01-17 12:43:17 CET 
When copying multiple "\r\n" from the remote session. only the first occurance of "\r\n" will be replaced with "\n".

We did not find a practical test-case for this, but by putting breakpoints in JavaScript, we could see that the clipboard text is incorrectly replaced.

This has been fixed upstream [1].

[1] https://github.com/novnc/noVNC/pull/1845
Comment 1 Adam Halim cendio 2025-01-20 13:46:59 CET 
Tested the same case as in comment #0 after the noVNC vendor drop and can confirm that all occurrences of "\r\n" now get replaced with "\n".
Note You need to log in before you can comment on or make changes to this bug.