metosin/reitit

[Feature] Add handlers registry

Ouverte

#695 ouverte le 26 août 2024

Voir sur GitHubGuide contributeur
 (2 commentaires) (1 réaction) (0 personne assignée)Clojure (261 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (1 569 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Just like middleware registry, but with a few open questions:

  • Should it be an opt of ring.core/router?
  • How should resolution be performed? Just postwalk and look for :handler?
    • Does that risk mangling input data?
    • Same for metadata (meta merge)
    • Can be done after merge but before compilation
  • Belong in core or ring?

Guide contributeur

Direction de recherche
Étudiez le registre middleware existant dans reitit et comprenez son implémentation. Ensuite, concevez un registre de gestionnaires suivant un modèle similaire, en abordant les questions ouvertes sur le placement, la résolution via postwalk et les impacts potentiels sur la structure des données.
Stack technique
clojure
Domaine
backend
Type d'issue
Fonctionnalité
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
ClojureGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65