remove instanceof checks and use reflection for admin cmd invocations
- 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