apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

Aperta
#6,966 5 commenti 0 reazioni 0 assegnatari Vedi su GitHub
no-issue-activity status:needs-work status:stale type:technical-debt
Lingua principale
Java
Stelle
3.1k
Fork
1.4k
Merge medio
6g 19h
PR unite (30g)
32

Descrizione

##### 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

~~~

~~~

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dalle classi API e dal livello di servizio che gestiscono le invocazioni dei comandi di admin, quindi confronta l’approccio di reflection descritto in #6951. Traccia i controlli instanceof esistenti per le istanze riservate esclusivamente al root admin e identifica i percorsi di invocazione interessati. Il lavoro è completato quando questi controlli utilizzano l’approccio di reflection previsto, preservando il comportamento riservato agli admin.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
api, cloud
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.