apache / apache/cloudstack

remove instanceof checks and use reflection for admin cmd invocations

Aberta
#6,966 5 comentários 0 reações 0 responsáveis Ver no GitHub
no-issue-activity status:needs-work status:stale type:technical-debt
Linguagem predominante
Java
Estrelas
3.1k
Forks
1.4k
Merge médio
6d 19h
PRs com merge (30d)
32

Descrição

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

~~~

~~~

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece pelas classes de API e pela camada de serviço que tratam das invocações de comandos de admin e, em seguida, compare a abordagem de reflection descrita em #6951. Rastreie as verificações instanceof existentes para instâncias exclusivas do root admin e identifique os caminhos de invocação afetados. Está concluído quando essas verificações usam a abordagem de reflection pretendida, preservando o comportamento exclusivo para admin.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
java
Domínio
api, cloud
Tipo de issue
Refatoração
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Pouca atividade
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.