Michael Pasternak has uploaded a new change for review.

Change subject: sdk: verify credentials before storing proxy in cache
......................................................................

sdk: verify credentials before storing proxy in cache

Change-Id: I7c38ae6f1830ec11fecaf8dca9cfa94e1c3b8e42
Signed-off-by: Michael Pasternak <[email protected]>
---
M src/codegen/entrypoint/entrypoint.py
M src/ovirtsdk/api.py
2 files changed, 6 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine-sdk refs/changes/48/7248/1

diff --git a/src/codegen/entrypoint/entrypoint.py 
b/src/codegen/entrypoint/entrypoint.py
index 9512fff..7b2e293 100644
--- a/src/codegen/entrypoint/entrypoint.py
+++ b/src/codegen/entrypoint/entrypoint.py
@@ -162,15 +162,15 @@
         # Create the proxy:
         proxy = Proxy(pool, persistent_auth)
 
-        # Store proxy to the context:
-        contextmanager.add('proxy', proxy, Mode.R)
-
         # Store entry point to the context
         contextmanager.add('entry_point',
                            proxy.request(method='GET',
                                          url='/api'),
                            Mode.R)
 
+        # Store proxy to the context:
+        contextmanager.add('proxy', proxy, Mode.R)
+
         # We need to remember if persistent auth is enabled:
         contextmanager.add('persistent_auth',
                            persistent_auth,
diff --git a/src/ovirtsdk/api.py b/src/ovirtsdk/api.py
index 63fc2b2..30aad76 100644
--- a/src/ovirtsdk/api.py
+++ b/src/ovirtsdk/api.py
@@ -82,15 +82,15 @@
         # Create the proxy:
         proxy = Proxy(pool, persistent_auth)
 
-        # Store proxy to the context:
-        contextmanager.add('proxy', proxy, Mode.R)
-
         # Store entry point to the context
         contextmanager.add('entry_point',
                            proxy.request(method='GET',
                                          url='/api'),
                            Mode.R)
 
+        # Store proxy to the context:
+        contextmanager.add('proxy', proxy, Mode.R)
+
         # We need to remember if persistent auth is enabled:
         contextmanager.add('persistent_auth',
                            persistent_auth,


--
To view, visit http://gerrit.ovirt.org/7248
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7c38ae6f1830ec11fecaf8dca9cfa94e1c3b8e42
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk
Gerrit-Branch: master
Gerrit-Owner: Michael Pasternak <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to