Bug 5190 - Remove Web Integration from tl-setup
Summary: Remove Web Integration from tl-setup
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Server Installer (show other bugs)
Version: 4.2.0
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.10.0
Assignee: Karl Mikaelsson
URL:
Keywords: hean01_tester, pcok, relnotes
Depends on: 4597
Blocks: 6082
  Show dependency treegraph
 
Reported: 2014-06-17 09:39 CEST by Peter Åstrand
Modified: 2018-09-13 16:06 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:
* Apache integration should not be installed with tl-setup * It should be possible to install the Apache integration by hand * The manual installation process and prerequisites should be described in the administrators guide


Attachments

Description Peter Åstrand cendio 2014-06-17 09:39:57 CEST
Currently, we let tl-setup create an Apache config file, so that tlclient.cgi can easily be deployed. When the Java Client Applet is removed, tlclient.cgi will only be used for portal integration of the native client. We could consider letting the customer configure Apache etc manually in case they want to use this functionality.
Comment 1 Pierre Ossman cendio 2016-11-15 10:34:30 CET
We need to make sure the documentation is sufficient to configure this by hand. Exactly how this needs to be structured is still an open question.
Comment 2 Pierre Ossman cendio 2018-07-03 11:27:00 CEST
Note the problems in bug 5706 and bug 5576 when documenting this.
Comment 3 Karl Mikaelsson cendio 2018-08-29 17:12:46 CEST
Bug 5916 will become obsolete by removing the auto-install steps from tl-setup. The root problem will still remain though, so it should probably still be mentioned in any new installation documentation.
Comment 4 Karl Mikaelsson cendio 2018-08-31 11:23:01 CEST
(In reply to comment #3)
> Bug 5916 will become obsolete by removing the auto-install steps from tl-setup.
> The root problem will still remain though, so it should probably still be
> mentioned in any new installation documentation.

I've changed my mind after discussing this with Henrik - I will not write anything in the documentation about configuring the firewall for the web server.
Comment 8 Henrik Andersson cendio 2018-09-13 11:02:22 CEST
> * Apache integration should not be installed with tl-setup
>

Installation of Web Integration is removed from tl-setup

> * It should be possible to install the Apache integration by hand
>
>
> * The manual installation process and prerequisites should be described in the administrators guide
>

Our documentation describes the requirements and installation process which fulfills the two acceptance criteria mentioned above.
Comment 9 Henrik Andersson cendio 2018-09-13 11:08:25 CEST
The script install-web-integration does not support an "upgrade", eg. if once run all subsequent calls to the script will fail with:

  Found Apache configuration directory: /etc/apache2/conf-enabled
  ERROR: /etc/apache2/conf-enabled/thinlinc.conf already exists!

  If you wish to install the configuration file to a different
  file name, set the environment variable APACHE_CONF_NAME to the
  desired file name and re-run this script. Example:

    env APACHE_CONF_NAME=thinlinc-portal-integration.conf /opt/thinlinc/share/web_integration/install-web-integration

This means if thinlinc.conf is changed with an release it will not be upgrade.

Possible solution is to always overwrite and add a note about upgrading in documentation.
Comment 12 Karl Mikaelsson cendio 2018-09-13 14:30:22 CEST
(In reply to comment #9)
> The script install-web-integration does not support an "upgrade", eg. if once
> run all subsequent calls to the script will fail with:
> 
>   Found Apache configuration directory: /etc/apache2/conf-enabled
>   ERROR: /etc/apache2/conf-enabled/thinlinc.conf already exists!
> 
>   If you wish to install the configuration file to a different
>   file name, set the environment variable APACHE_CONF_NAME to the
>   desired file name and re-run this script. Example:
> 
>     env APACHE_CONF_NAME=thinlinc-portal-integration.conf
> /opt/thinlinc/share/web_integration/install-web-integration
> 
> This means if thinlinc.conf is changed with an release it will not be upgrade.
> 
> Possible solution is to always overwrite and add a note about upgrading in
> documentation.

Fixed by r33666.
Comment 14 Henrik Andersson cendio 2018-09-13 16:06:12 CEST
Finally verified that release notes which also looks good.

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