Bug 1077448 - [ovirt][webadmin] SessionID for REST API stores in browser Local Storage
Summary: [ovirt][webadmin] SessionID for REST API stores in browser Local Storage
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: 3.4
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: 3.4.1
Assignee: Alexander Wels
QA Contact: Pavel Stehlik
URL:
Whiteboard: ux
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-03-18 03:26 UTC by lzhuang
Modified: 2014-05-08 13:36 UTC (History)
14 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-05-08 13:36:55 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 25987 0 None None None Never
oVirt gerrit 26807 0 ovirt-engine-3.4 MERGED webadmin: Store rest api session id in http session Never

Description lzhuang 2014-03-18 03:26:28 UTC
Description of problem:
oVirt stores session ID of REST API(https://<host_ip>/ovirt-engine/api) in Local Storage. An attacker could stole the session ID via a single XSS attack.

Version-Release number of selected component (if applicable):
oVirt 3.4.0-5 beta3

How reproducible:
100%

Steps to Reproduce:
1. Login to https://<host_ip>/ovirt-engine/webadmin
2. Open browser debugger and check "Local Storage"

Actual results:
oVirt stores session ID of REST API in Local Storage

Expected results:
Sensitive data should not be stored in Local Storage, controls to treat untrusted data in Local Storage are also needed.

Additional info:

Comment 1 Kurt Seifried 2014-03-24 19:15:58 UTC
When you say "local storage" do you mean a local cookie, HTML5 storage, or something else?

Comment 2 Einav Cohen 2014-03-24 20:12:50 UTC
(In reply to Kurt Seifried from comment #1)
> When you say "local storage" do you mean a local cookie, HTML5 storage, or
> something else?

probably both, but I would like lzhuang to confirm that. lzhuang?

Comment 3 Alexander Wels 2014-03-24 20:17:08 UTC
I believe the issue lzhuang is revering to is HTML5 storage. For browsers that don't support this (IE8) we fall back to using a cookie.

Comment 4 lzhuang 2014-03-25 01:33:02 UTC
(In reply to Einav Cohen from comment #2)
> (In reply to Kurt Seifried from comment #1)
> > When you say "local storage" do you mean a local cookie, HTML5 storage, or
> > something else?
> 
> probably both, but I would like lzhuang to confirm that. lzhuang?

Hi,

"local storage" means HTML5 storage. As described in OWASP:
Local Storage also known as Web Storage or Offline Storage is a mechanism to store data as key/value pairs tied to a domain and enforced by the same origin policy (SOP).

Comment 6 Sandro Bonazzola 2014-05-08 13:36:55 UTC
This is an automated message

oVirt 3.4.1 has been released:
 * should fix your issue
 * should be available at your local mirror within two days.

If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.