Project

General

Profile

Actions

Bug #26494

open

Inventory tables takes significant storage leading to QuotaExceededError with many nodes

Added by Clark ANDRIANASOLO 4 days ago. Updated 4 days ago.

Status:
New
Priority:
To review
Assignee:
-
Category:
Web - Maintenance
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
It bothers me each time
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Priority:
95
Name check:
To do
Fix check:
To do
Regression:
No

Description

When using the technique editor, I could end-up not being able to edit drafts, and I receive an error notification with a QuotaExceededError in the details, suggesting that I'm out of storage in the browser.

This is likely due to accumulated data in localStorage : when consulting the node details page of a node for the first time, opening the page leads to several data added to the localstorage, corresponding to an empty state of the bios, processors, net, var, os, storages, process, logsGrid, fs, softUpdates, controllers, etc.
All data is added in batch, this is due to using the DataTables library and displaying all tables as soon as the page is rendered, even if the "Inventory" tab and specific table has not been clicked in the browser.

Proposed solution :
We should make this lazy, at least by rendering the table data with AJAX instead of rendering it on the server-side page : https://github.com/Normation/rudder/blob/branches/rudder/8.1/webapp/sources/rudder/rudder-web/src/main/scala/com/normation/rudder/web/services/DisplayNode.scala#L420.

We could also do storage optimization in https://github.com/Normation/rudder/blob/branches/rudder/8.1/webapp/sources/rudder/rudder-web/src/main/javascript/rudder/rudder-datatable.js#L2423, but it could lead to compatibility issues and duplication, especially if we want to migrate away from DataTables.

Workaround is to empty the localStorage : localStorage.clear() in the browser console.


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #6157: Rudder stores too much or too big cookies, causing white page with HTTP status: 413: FULL head (branch 2.11)ReleasedFrançois ARMAND2015-01-20Actions
Actions #1

Updated by Clark ANDRIANASOLO 4 days ago

  • UX impact set to It bothers me each time
  • User visibility set to Getting started - demo | first install | Technique editor and level 1 Techniques
  • Priority changed from 0 to 62
Actions #2

Updated by Elaad FURREEDAN 4 days ago

  • Priority changed from 62 to 95
Actions #3

Updated by Clark ANDRIANASOLO 4 days ago

  • Related to Bug #6157: Rudder stores too much or too big cookies, causing white page with HTTP status: 413: FULL head (branch 2.11) added
Actions #4

Updated by Clark ANDRIANASOLO 4 days ago

  • Priority changed from N/A to To review
  • Priority changed from 95 to 62
Actions #5

Updated by Clark ANDRIANASOLO 4 days ago

  • Priority changed from 62 to 95
Actions

Also available in: Atom PDF