URL: https://github.com/freeipa/freeipa/pull/328
Author: mbasti-rh
 Title: #328: fix: regression in API version comparison
Action: opened

PR body:
"""
Commint 2cbaf156045769b54150e4d4c3c1071f164a16fb introduced a regression
in API version comparison. In case that newer client is trying to call
older server an error is returned, but it should work. This commit fixes
it.

https://fedorahosted.org/freeipa/ticket/6468
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/328/head:pr328
git checkout pr328
From 36dfd4524acb3d3ab595c3ea126035f8781a451f Mon Sep 17 00:00:00 2001
From: Martin Basti <mba...@redhat.com>
Date: Mon, 12 Dec 2016 18:28:53 +0100
Subject: [PATCH] fix: regression in API version comparison

Commint 2cbaf156045769b54150e4d4c3c1071f164a16fb introduced a regression
in API version comparison. In case that newer client is trying to call
older server an error is returned, but it should work. This commit fixes
it.

https://fedorahosted.org/freeipa/ticket/6468
---
 ipalib/frontend.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/ipalib/frontend.py b/ipalib/frontend.py
index dfa6cdf..41dcd4a 100644
--- a/ipalib/frontend.py
+++ b/ipalib/frontend.py
@@ -776,8 +776,7 @@ def verify_client_version(self, client_version):
                                sver=self.api_version,
                                server=self.env.xmlrpc_uri)
 
-        if (client_apiver.major != server_apiver.major
-                or client_apiver > server_apiver):
+        if client_apiver.major != server_apiver.major:
             raise VersionError(cver=client_version,
                                sver=self.api_version,
                                server=self.env.xmlrpc_uri)
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to