Project

General

Profile

Actions
Copy link

User story #14194

closed

Manage caching of technique editor resources (css/js) with Rudder version

Added by Vincent MEMBRÉ almost 6 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - Technique editor
Target version:
4.1.19
Pull Request:
https://github.com/Normation/ncf/pull...
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Currently when you upgrade your Rudder, ncf could be out of date due to browser caching of web resources, since our js/css files are not cached.

One way to manage this is to add a parameter to url which would be set with the version of Rudder

ie:

instead of having:

<script src="/js/ncf.js"></script>

We should have:

<script src="/js/ncf.js?version=x.y.z"></script>

We should set version to dev on the source code, and only replace 'dev' by the version at build time when we know the valid version (could also be at tag time but would prevent to do this on nightlies ...)

Subtasks 2 (0 open — 2 closed)

User story #14195: Set version in url parameters of js/css resource of technique editorReleasedVincent MEMBRÉActions
Bug #14212: In technique editor, ncf index is cached which defeats #14194 correction on migrationReleasedVincent MEMBRÉActions

Related issues 1 (0 open — 1 closed)

Related to Rudder - User story #14210: Missing headers to cache technique editor css/js resourcesRejectedActions
Actions
Copy link
 #1

Updated by Vincent MEMBRÉ almost 6 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Vincent MEMBRÉ almost 6 years ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ almost 6 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
  • Pull Request set to https://github.com/Normation/ncf/pull/906
Actions
Copy link
 #4

Updated by Rudder Quality Assistant almost 6 years ago

  • Assignee changed from François ARMAND to Vincent MEMBRÉ
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ almost 6 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:8b6103b00d4b81d4d3ee97fd21b5010ad145e952.

Actions
Copy link
 #6

Updated by François ARMAND almost 6 years ago

  • Related to User story #14210: Missing headers to cache technique editor css/js resources added
Actions
Copy link
 #7

Updated by Alexis Mousset almost 6 years ago

  • Status changed from Pending release to Released
This bug has been fixed in Rudder 4.1.19, 4.3.9 and 5.0.5 which were released today. 
Changelog
 Changelog
 Changelog
Actions
Copy link
 #8

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - UI/UX to Web - Technique editor
Actions
Copy link

Also available in: Atom PDF