alibaba/Sentinel

Dashboard Spring Boot Configuration more flexible support

Open

#2 053 ouverte le 27 févr. 2021

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Java (23 109 stars) (8 150 forks)batch import
area/dashboardgood first issuekind/enhancement

Description

Issue Description

Type: feature request

Describe what happened (or what feature you want)

Make dashboard more extensible.

Describe what you expected to happen

Spring Boot Configuration more flexible

Use @ConditionalOnMissingBean not @Component directly, combine with @Configuration to create a bean.

For example, about MachineDiscovery.

We can write code like follow

@Configuration
public class MachineDiscoveryConfiguration {

  @Bean
  @ConditionalOnMissingBean
  public SimpleMachineDiscovery simpleMachineDiscovery() {
    return new SimpleMachineDiscovery();
  }

  @Bean
  @ConditionalOnMissingBean
  public AppManagement appManagement(SimpleMachineDiscovery simpleMachineDiscovery) {
    return new AppManagement(simpleMachineDiscovery);
  }

}

And delete annotation @Component above class SimpleMachineDiscovery and AppManagement.

If someone want to custom their AppManagement, that will make it more easy.

Same principle can apply to RuleRepository, DynamicRuleProvider, DynamicRulePublisher too.

Guide contributeur

Stack technique
javaspring
Domaine
backend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
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
JavaSpring Boot basicsDependency injection
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
Replace @Component annotations on SimpleMachineDiscovery and AppManagement with a @Configuration class containing @Bean methods annotated with @ConditionalOnMissingBean. Apply the same pattern to RuleRepository, DynamicRuleProvider, and DynamicRulePublisher. Ensure that existing behavior is preserved and custom beans can be overridden.