[PATCH] bug fix: Add authentication to host tab

6 views
Skip to first unread message

Aline Manera

unread,
Dec 17, 2013, 12:12:20 PM12/17/13
to Project Kimchi
From: Aline Manera <ali...@br.ibm.com>

The host tab must be visible only for logged users.
So add authentication to it.

Signed-off-by: Aline Manera <ali...@br.ibm.com>
Signed-off-by: Malcolm Yu <minghao...@gmail.com>
---
src/kimchi/server.py | 1 +
ui/js/src/kimchi.api.js | 2 ++
2 files changed, 3 insertions(+)

diff --git a/src/kimchi/server.py b/src/kimchi/server.py
index 70c6859..6ff6fa0 100644
--- a/src/kimchi/server.py
+++ b/src/kimchi/server.py
@@ -73,6 +73,7 @@ class Server(object):
'tools.sessions.storage_type': 'file',
'tools.sessions.storage_path': config.get_session_path(),
'tools.kimchiauth.on': False},
+ '/host': {'tools.kimchiauth.on': True},
'/vms': {'tools.kimchiauth.on': True},
'/templates': {'tools.kimchiauth.on': True},
'/networks': {'tools.kimchiauth.on': True},
diff --git a/ui/js/src/kimchi.api.js b/ui/js/src/kimchi.api.js
index aa1a304..fbcf4a2 100644
--- a/ui/js/src/kimchi.api.js
+++ b/ui/js/src/kimchi.api.js
@@ -71,6 +71,7 @@ var kimchi = {
kimchi.requestJSON({
url : kimchi.url + 'host',
type : 'GET',
+ resend: true,
contentType : 'application/json',
dataType : 'json',
success : suc,
@@ -85,6 +86,7 @@ var kimchi = {
kimchi.requestJSON({
url : kimchi.url + 'host/stats',
type : 'GET',
+ resend: true,
contentType : 'application/json',
dataType : 'json',
success : suc,
--
1.7.10.4

Ramon Medeiros

unread,
Dec 17, 2013, 12:15:01 PM12/17/13
to Aline Manera, Project Kimchi
Reviewed-By: Ramon Medeiros <ram...@br.ibm.com>
Ramon Nunes Medeiros
Software Engineer - Linux Technology Center Brazil
IBM Systems & Technology Group
Phone : +55 19 2132 7878
ram...@br.ibm.com

Daniel Henrique Barboza

unread,
Dec 17, 2013, 1:31:48 PM12/17/13
to project...@googlegroups.com
Reviewed-by: Daniel Barboza <dani...@linux.vnet.ibm.com>

Aline Manera

unread,
Dec 17, 2013, 2:08:43 PM12/17/13
to Aline Manera, Project Kimchi
Applied. Thanks.

Regards,

Aline Manera

Reply all
Reply to author
Forward
0 new messages