remove instanceof checks and use reflection for admin cmd invocations
- Lenguaje dominante
- Java
- Estrellas
- 3.1k
- Forks
- 1.4k
- Merge medio
- 6 d 19 h
- PR fusionados (30 d)
- 32
Descripción
##### 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
~~~
~~~
Guía de contribución
Línea de trabajo
Comienza en las clases de API y la capa de servicios que gestionan las invocaciones de comandos de admin, y después compara el enfoque de reflection descrito en #6951. Traza las comprobaciones instanceof existentes para las instancias exclusivas del root admin e identifica las rutas de invocación afectadas. Se considera terminado cuando esas comprobaciones utilizan el enfoque de reflection previsto y conservan el comportamiento exclusivo para admin.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- api, cloud
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100