Hello Yedidyah Bar David,
I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/20442
to review the following change.
Change subject: packaging: setup: clean '&' in jdbcTlsOptions
......................................................................
packaging: setup: clean '&' in jdbcTlsOptions
Without this change, we might get an '&' immediately after the '?' in
ENGINE_DB_URL.
Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1002401
Change-Id: If8bb485e03c9d9d4c967f763a157151b11cbb84d
Signed-off-by: Yedidyah Bar David <[email protected]>
---
M packaging/setup/plugins/ovirt-engine-setup/config/database.py
1 file changed, 15 insertions(+), 12 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/42/20442/1
diff --git a/packaging/setup/plugins/ovirt-engine-setup/config/database.py
b/packaging/setup/plugins/ovirt-engine-setup/config/database.py
index 542441a..206bd23 100644
--- a/packaging/setup/plugins/ovirt-engine-setup/config/database.py
+++ b/packaging/setup/plugins/ovirt-engine-setup/config/database.py
@@ -89,19 +89,22 @@
securedValidation=self.environment[
osetupcons.DBEnv.SECURED_HOST_VALIDATION
],
- jdbcTlsOptions='%s%s' % (
+ jdbcTlsOptions='&'.join(
+ s for s in (
+ 'ssl=true'
+ if self.environment[
+ osetupcons.DBEnv.SECURED
+ ] else '',
- 'ssl=true'
- if self.environment[
- osetupcons.DBEnv.SECURED
- ] else '',
-
- '&sslfactory=org.postgresql.ssl.NonValidatingFactory'
- if not self.environment[
- osetupcons.DBEnv.SECURED_HOST_VALIDATION
- ] else ''
-
- )
+ (
+ 'sslfactory='
+ 'org.postgresql.ssl.NonValidatingFactory'
+ )
+ if not self.environment[
+ osetupcons.DBEnv.SECURED_HOST_VALIDATION
+ ] else ''
+ ) if s
+ ),
),
modifiedList=self.environment[
otopicons.CoreEnv.MODIFIED_FILES
--
To view, visit http://gerrit.ovirt.org/20442
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If8bb485e03c9d9d4c967f763a157151b11cbb84d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
