apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

Ouverte
#6,966 5 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
no-issue-activity status:needs-work status:stale type:technical-debt
Langage dominant
Java
Étoiles
3.1k
Forks
1.4k
Merge moyen
6 j 19 h
PR mergées (30 j)
32

Description

##### ISSUE TYPE
* cleanup

##### COMPONENT NAME

~~~
API
~~~

##### CLOUDSTACK VERSION

~~~
4.17
~~~

##### CONFIGURATION

##### OS / ENVIRONMENT

##### SUMMARY

the API classes have instances that are for root admin only, these are checked for using `instanceof` in the service layer. This should be cleaned up in favour for reflection invocations as in #6951. It would be better to do all in one go but the fix in #6951 really needs it and is for a point release, so postponing.

##### STEPS TO REPRODUCE

~~~

~~~

##### EXPECTED RESULTS

~~~

~~~

##### ACTUAL RESULTS

~~~

~~~

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par les classes d’API et la couche de service qui gèrent les invocations de commandes d’admin, puis comparez l’approche de reflection décrite dans #6951. Suivez les vérifications instanceof existantes pour les instances réservées au root admin et identifiez les chemins d’invocation concernés. C’est terminé lorsque ces vérifications utilisent l’approche de reflection prévue tout en conservant le comportement réservé aux admins.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
api, cloud
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

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