simonw/datasette

sql-formatter javascript is not now working with CloudFlare rocketloader

Open

#1 969 ouverte le 23 déc. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (8 637 stars) (625 forks)batch import
bughelp wantedjavascript

Description

This is probably not a bug with datasette, but I thought you might want to know, @simonw.

I noticed today that my CloudFlare proxied datasette instance lost the "Format SQL" option. I'm pretty sure it was there last week.

In the CloudFlare settings, if I turn off Rocket Loader, I get the "Format SQL" option back.

Rocket Loader works by asynchronously loading the javascript, so maybe there was a recent change that doesn't play well with the asynch loading?

I'm up to date with https://github.com/simonw/datasette/commit/e03aed00026cc2e59c09ca41f69a247e1a85cc89

Guide contributeur

Stack technique
javascript
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic JavaScriptKnowledge of CloudFlare Rocket Loader
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Investigate how the JavaScript for the SQL formatter is loaded in datasette. Look at the relevant commit e03aed00026cc2e59c09ca41f69a247e1a85cc89 to see if any changes were made to script loading. Check if the script has an 'async' or 'defer' attribute or if it is loaded via a dynamic import that conflicts with Rocket Loader. Consider adding a script attribute or ensuring the script loads synchronously when Rocket Loader is active.