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