Bug 6082 - remove web integration (tlclient.cgi)
Summary: remove web integration (tlclient.cgi)
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: VSM Server (show other bugs)
Version: 1.3.1
Hardware: PC Unknown
: P2 Normal
Target Milestone: MediumPrio
Assignee: Henrik Andersson
URL:
Keywords:
Depends on: 4521 5190
Blocks:
  Show dependency treegraph
 
Reported: 2016-11-01 15:16 CET by Pierre Ossman
Modified: 2020-02-25 13:04 CET (History)
2 users (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2016-11-01 15:16:04 CET
It relies on Java for a lot of its functionality, we believe it is lightly used, and many customers confuse it with the HTML client. As such the downsides seem to cancel out any benefit.
Comment 1 Pierre Ossman cendio 2016-11-01 15:22:34 CET
The product council approves of removing this functionality. Two requirements though:

 - An attempt should be made to retain shadowing in tlwebadm, provided it doesn't require a massive effort.

 - The removed code should be donated to https://github.com/oetiker/tl-extras, if they are interested
Comment 2 Samuel Mannehed cendio 2016-11-01 15:33:15 CET
(In reply to comment #1) 
>  - An attempt should be made to retain shadowing in tlwebadm, provided it
> doesn't require a massive effort.

Bug 4928 covers implementing shadowing support for the HTML5 client.
Comment 3 Karl Mikaelsson cendio 2016-11-07 17:09:53 CET
(In reply to comment #2)
> (In reply to comment #1) 
> >  - An attempt should be made to retain shadowing in tlwebadm, provided it
> > doesn't require a massive effort.
> 
> Bug 4928 covers implementing shadowing support for the HTML5 client.

Shadowing does not have to be implemented in the HTML5 client for this. We could keep the native client configuration file generation function and move it into tlwebadm.
Comment 4 Pierre Ossman cendio 2016-11-08 14:07:10 CET
Completely removing this feature is probably premature. But we can mitigate some of the most glaring problems:

 - No Java these days (bug 4521)

 - Confusion in tl-setup (bug 5190)

Note You need to log in before you can comment on or make changes to this bug.