apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

Abierto
#6,966 5 comentarios 0 reacciones 0 asignados Ver en GitHub
no-issue-activity status:needs-work status:stale type:technical-debt
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.