Project

General

Profile

Actions

Feature #1913

closed

Move winrm truststore path setting to IdM

Added by Petr Fišer about 5 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Roman Kučera
Target version:
Start date:
10/24/2019
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Currently, a path to the truststore is hardcoded in the https://git.bcvsolutions.eu/modules/winrm-ad-connector/blob/master/scripts/winrm_wrapper.py#L90 ca_trust_path setting.
This can cause problems when deploying to other paths than expected. I also accidentally clashed into this because of deployment guide rewrite. :)
After discussing with Roman, we agreed upon moving the truststore path into system configuration in IdM. The truststore path will be given to the winrm_wrapper.py script in a variable directly from the IdM.

This is an implementation ticket for the change.

Another option is to set some environment variable of the connector server user and read it from python like:

import os
truststorepath = os.environ["envvariablename"]

Preferred way is to supply truststore path directly from the IdM. Then, the winrm_wrapper.py script will be deployment-agnostic and also whole configuration will be stored in one place in the IdM.

Actions #1

Updated by Roman Kučera over 4 years ago

  • Target version changed from version 1.0.3 to version 1.0.4
Actions #2

Updated by Roman Kučera over 4 years ago

  • Assignee set to Roman Kučera
Actions #3

Updated by Roman Kučera over 4 years ago

  • Target version changed from version 1.0.4 to version 1.0.5
Actions #4

Updated by Roman Kučera over 4 years ago

  • % Done changed from 0 to 80

Implemented in https://github.com/bcvsolutions/winrm-ad-connector/tree/1913-trustore-path-in-config
Together with this I added option for skipping CA validation to config to.
Change all template scripts to be compatible with new params in wrapper.

Actions #5

Updated by Roman Kučera over 4 years ago

  • % Done changed from 80 to 90

I test it on local env and its working as intended

Actions #6

Updated by Tomáš Doischer about 4 years ago

Code review done, looks good.

Please update documentation at least here: https://wiki.czechidm.com/devel/documentation/adm/systems/winrm_ad_connector

Actions #8

Updated by Roman Kučera about 4 years ago

  • Status changed from New to Closed
  • % Done changed from 90 to 100

relesed

Actions

Also available in: Atom PDF