Sprints: Impossible to edit any page of MDN

Created on 14 Dec 2018  路  9Comments  路  Source: mdn/sprints

Steps to reproduce:

  1. Login into MDN
  2. Open any article for edition

Current result (reproduced in release Fx/Edge/Chrome):
No editor is displayed. It's impossible to edit the content.
3 errors appear in the console

Expected result:
CKeditor loads as usual and a contributor is able to edit a page.

Most helpful comment

The build is fine, as shown by it working in development and staging. The issue is that the production CDN was returning last week's version of this file:

https://developer.mozilla.org/static/js/libs/ckeditor/build/ckeditor/lang/en.js?t=IBDI

CKEditor uses the build timestamp (t=IBDI) for cache breaking, but our CDN discards all query string parameters for the /static path. It appears we need an exception for t= for the CKEditor build folder.

The CDN is composed of many caching servers distributed globally. My guess is that some people were able to edit because their CDN had the more recent copy of en.js.

If we waited a week, editing would work for everyone. However, in the holiday spirit, I told the CDN to invalidate the cache for this folder and download fresh files. Editing should now work for everyone.

All 9 comments

developer.mozilla.org-1544770319746.log
Here is the log I got from Chrome

And of course it works fine when running locally ;p - Makes me think it has something to do with the build maybe? The file that is failing to load is in the bundle. What I do notice however is that the source is located at:

kuma/static/js/libs/ckeditor/source/plugins/a11ychecker/lang/en.js

And the build is at:

kuma/static/js/libs/ckeditor/build/ckeditor/lang/en.js

The one it is failing to load is at:

/static/js/libs/ckeditor/build/ckeditor/plugins/a11ychecker/lang/en.js?t=IBDI

Wondering if this is what is needed:
https://kuma.readthedocs.io/en/latest/ckeditor.html#building-ckeditor

Yeah, it sounds like the editor didn't get rebuilt ant its files checked in, maybe.

It looks like a11yhelp got added to the build, but not a11ychecker.

Weirdly enough, it looks like some people are still managing to make edits.

The build is fine, as shown by it working in development and staging. The issue is that the production CDN was returning last week's version of this file:

https://developer.mozilla.org/static/js/libs/ckeditor/build/ckeditor/lang/en.js?t=IBDI

CKEditor uses the build timestamp (t=IBDI) for cache breaking, but our CDN discards all query string parameters for the /static path. It appears we need an exception for t= for the CKEditor build folder.

The CDN is composed of many caching servers distributed globally. My guess is that some people were able to edit because their CDN had the more recent copy of en.js.

If we waited a week, editing would work for everyone. However, in the holiday spirit, I told the CDN to invalidate the cache for this folder and download fresh files. Editing should now work for everyone.

Confirming it works now on my "edge"

Was this page helpful?
0 / 5 - 0 ratings