v4 'cachePath' with absolute path

Ouverte
  • Quelque chose ne va pas ?
#562 6 commentaires 1 réaction 1 personne assignée Voir sur GitHub

@jszuminski y travaille déjà.

Depuis le 2/9/2024.

Évaluation

Cette issue n'a pas encore été évaluée.

Description

enhancement
Expected behaviour

highcharts.cachePath or HIGHCHARTS_CACHE_PATH env var should allow an absolute path such as '/tmp/.cache'

Actual behaviour

Previously (v3) if the manifest.json file existed in the 'node_modules/highcharts-export-server/.cache' directory but read-only it would log an error and initPool would continue to start. Now initExport throws an error.

Reproduction steps

Set the 'node_modules/highcharts-export-server/.cache/manifest.json' file as read-only and initExport will throw an error.
Ideally just allow cachePath to specify an absolute path and if the manifest.json file exists (and un-changed) then skip writing new manifest

Langage dominant
JavaScript
Étoiles
368
Forks
266
Aucune PR mergée en 30 j

Préparer son environnement

  • Fournit un Dockerfile ou un fichier Docker Compose
  • Aucun modèle de pull request
  • Aucun guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de highcharts/node-export-server

Toutes les issues de highcharts/node-export-server

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.