Relancer un projet obsolète

Cette semaine, j'ai effectué quelques travaux de maintenance sur Starchart. Le projet n'a pas été travaillé depuis un moment, nous essayons donc de mettre à jour ses dépendances.

DéveloppementEspace / diagramme d'amidon

Un outil en libre-service pour gérer les domaines et certificats personnalisés

Carte des étoiles

Starchart permet à la communauté des développeurs Seneca de créer et de gérer facilement ses propres sous-domaines personnalisés et certificats SSL, sans frais ni devoir fournir d'informations personnelles.

Pour plus d'informations sur l'exécution de Starchart, consultez notre guide de déploiement. Pour plus d’informations sur le développement, consultez notre guide de contribution. Pour plus d'informations techniques, de planification et de conceptions initiales, veuillez consulter le wiki.

Présentation

Internet évolue et ce qui était difficile est devenu simple. Par exemple, héberger votre propre site Web nécessitait auparavant des connaissances en administration de serveur, en systèmes d'exploitation, en réseau, etc. Aujourd'hui, de nombreux développeurs hébergent leurs sites Web personnels et de projet sans jamais toucher à un serveur distant, optant pour des services cloud (gratuits) comme GitHub Pages, Vercel, Netlify ou AWS.

Le modèle de sécurité d'Internet évolue également. Par exemple, les fournisseurs de navigateurs ont adopté HTTPS partout. C'est bon pour la sécurité, car cela permet un cryptage basé sur des certificats entre les clients et les serveurs. Cependant, comme avec…

Voir sur GitHub

Le plan était de réparer le flux de travail CI, dont nous avons découvert qu'il était défectueux la semaine dernière :

Mais avant que je puisse trouver une solution, l'un des développeurs précédents, Eakam, a résolu le problème - il s'est avéré que c'était simplement parce que Playwright était obsolète.

Faire passer le dramaturge à la version 1.49.1 #772

Eakam1007 publié le 18 janvier 2025

L'installation de Playwright échoue dans CI (tests E2E). La version du dramaturge devrait résoudre ce problème.

Réf :

Voir sur GitHub

J'avais l'impression que je devrais compenser cela en trouvant plus de choses sur lesquelles travailler et j'ai pensé que mettre à jour davantage de dépendances serait un excellent point de départ.

Comme le projet n'avait pas été travaillé depuis 2 ans, il y avait un tas de failles de sécurité provenant de packages obsolètes. J'ai pu réparer la plupart d'entre eux avec npm audit fix.

Il y a eu quelques correctifs supplémentaires qui ont conduit à des modifications importantes dans @remix-run/eslint-config et @remix-run/react, donc je les ai corrigés manuellement.

L'une des mises à jour (je les ai cognées en même temps donc je ne peux pas le dire avec certitude mais mon pari est sur /react) a conduit à une erreur de vérification de type car [@remix-run/react].useNavigation( ).formData peut désormais être du type non défini. Je l'ai corrigé avec un chaînage optionnel.

// Before
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData.get('id')) === dnsRecord.id;

// After
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData?.get('id')) === dnsRecord.id;

Les autres modifications que j'ai apportées étaient liées à des erreurs de charpie qui sont apparues (à ce stade, j'ai réalisé que mon extension ESLint était désactivée, mais je suis sûr que ces avertissements sont venus avec la mise à jour, car cela ne s'est jamais produit dans CI dans le passé).

1. Instances d'importation du même module plusieurs fois dans un seul fichier :

// Before
import { getCertificateByUsername } from '~/models/certificate.server';
import { deleteCertificateById } from '~/models/certificate.server';
import { isAdmin } from '~/models/user.server';
import { getUserByUsername } from '~/models/user.server';

// After
import { getCertificateByUsername, deleteCertificateById } from '~/models/certificate.server';
import { isAdmin, getUserByUsername } from '~/models/user.server';

1. Utiliser let lorsque const est préféré :

// Before
let date = val.toLocaleDateString('en-US', {

// After
const date = val.toLocaleDateString('en-US', {

Surpris qu'il ne les ait pas détectés avant.

De plus, lorsque j'ai activé l'extension ESLint, j'ai été un peu surpris car il y avait environ 900 erreurs de linter. Il s'est avéré que c'était parce qu'ESLint peluchait la sortie générée par Playwright. J'ai donc ajouté /playwright-report à .eslintignore.

Et c'était la somme de mon travail de maintenance pour ce sprint. J'ai fini par résoudre 30 problèmes de sécurité graves, donc pas mal.

Mettre à jour les dépendances #775

ouday-rana publié le 18 janvier 2025

Devrait corriger un tas de vulnérabilités de sécurité.

Modifications

• [x] Dépendances de bosse
• [x] Ajouter /playwright-report à .eslintignore
• [x] Correction des erreurs de vérification de type et de linter

Voir sur GitHub

J'ai également réactivé Dependabot, ce qui a modifié quelques versions mineures. Ce serait bien de ne pas avoir à enquêter et à corriger manuellement les vulnérabilités de sécurité.

Par ailleurs, une de mes pull request vers Mattermost a finalement été fusionnée !

[GH-29548] Évitez SELECT * dans `tokens_store.go` #29558

ouday-rana publié le 10 décembre 2024

Résumé

Ce PR :

• Bascule les requêtes SQL dans token_store.go pour utiliser SQLBuilder
• Définit explicitement les colonnes dans les requêtes SELECT vers TokenStore.
• Factifie les requêtes courantes dans le constructeur.

Lien du billet

Corrections #29548

Captures d'écran

Note de version

// Before
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData.get('id')) === dnsRecord.id;

// After
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData?.get('id')) === dnsRecord.id;

Voir sur GitHub

Il avait été approuvé il y a quelque temps mais il a fallu quelques semaines pour être fusionné avec main.

En attendant, je travaille sur mes autres relations publiques. On m'a demandé d'apporter quelques modifications et j'attends une nouvelle révision.

[MM-53650] Ajouter un paramètre de désactivation du rendu des émoticônes à l'application Web #29414

ouday-rana publié le 28 novembre 2024

Résumé

Cette pull request ajoute un paramètre utilisateur à l'application Web pour basculer le rendu des émoticônes (:D) sous forme d'emojis (?).

Le paramètre est ajouté en tant que composant dans Components/user_settings/display/render_emoticons_as_emoji/ qui est importé dans Components/user_settings/display/user_settings_display.tsx.

J'ai ajouté une fonction renderOnOffLabel() à user_settings_display.tsx, extraite de components/user_settings/advanced/user_settings_advanced.tsx pour aider au rendu du nouveau composant.

Le paramètre est stocké en tant que préférence utilisateur à l'aide de l'action savePreferences().

J'ai ajouté des constantes pour la préférence à utils/constants.tsx et webapp/channels/src/packages/mattermost-redux/src/constants/preferences.ts.

Pour utiliser réellement le paramètre, j'ai modifié composants/post_markdown pour recevoir sa valeur en tant qu'accessoire, pour lequel j'ai utilisé getBool() et ajouté une valeur par défaut à la configuration. post_markdown transmet cette valeur à Markdown sur l'objet options, qui la transmet ensuite à utils/text_formatting.tsx, qui transmet finalement la valeur à emoticons.tsx en tant que paramètre nouvellement ajouté. emoticons.tsx vérifie si la valeur est vraie et si c'est le cas, il transforme les émoticônes en emojis.

J'ai mis à jour les tests concernés et créé des tests unitaires pour le nouveau composant. J'ai également mis à jour le fichier de traduction en anglais.

Étapes du test QA

1. Accédez aux paramètres utilisateur.
2. Allez dans la catégorie Affichage.
3. Recherchez la section intitulée « Rendu automatique des émoticônes en tant qu'emoji » et cliquez sur « Modifier ».
4. Basculez le paramètre et cliquez sur "Enregistrer".
5. Le rendu des émoticônes sur les messages envoyés par l'utilisateur actuel et les autres utilisateurs doit être activé côté client avec le paramètre.

Lien du billet

Corrections (partiellement) https://github.com/mattermost/mattermost/issues/26504 Jira https://mattermost.atlassian.net/browse/MM-53650

Remarque le problème et le ticket décrivent également l'ajout de cette fonctionnalité à l'application mobile, ce que ce PR ne fait pas.

Captures d'écran

before	after

Note de version

// Before
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData.get('id')) === dnsRecord.id;

// After
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData?.get('id')) === dnsRecord.id;

Voir sur GitHub

Travailler sur ce PR était intéressant car lorsque je l'ai soumis pour la première fois, je n'avais même pas entièrement compris mes modifications. Y revenir après une longue absence et grâce aux retours des critiques m'a aidé à le regarder sous un nouvel angle et à mieux le comprendre.

L'application Mattermost obtient l'état des paramètres utilisateur à partir des « préférences » et d'une « configuration ». J'ai ajouté mon paramètre aux deux, imitant l'un des paramètres existants qu'il m'a été conseillé de référencer, mais il s'est avéré que la "config" est destinée aux paramètres au niveau du serveur, alors que ce nouveau paramètre était destiné à être une option côté client. Les critiques m'ont aidé à comprendre où je m'étais trompé, et cela a fini par être un changement moindre que ce que je pensais nécessaire.

Dans l'ensemble, je dirais que ce fut une semaine assez productive.

Ce qui précède est le contenu détaillé de. pour plus d'informations, suivez d'autres articles connexes sur le site Web de PHP en chinois!