reactioncommerce/reaction

Surchages query missing permission validation

Aberta

#6.634 aberto em 7 de nov. de 2022

Ver no GitHubGuia do colaborador
 (9 comentários) (0 reação) (1 responsável)JavaScript (2.198 forks)batch import
buggood first issueneeds triage

Métricas do repositório

Stars
 (12.181 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Prerequisites

  • Are you running the latest version?
  • Are you able to consistently reproduce the issue?
  • Did you search the issue queue for existing issue? Search issues

Issue Description

The surcharges query in api-plugin-surcharges is missing the read permission validation. https://github.com/reactioncommerce/reaction/blob/b11e47f05a3d3042b76385e992960dfafb36a286/packages/api-plugin-surcharges/src/queries/surcharges.js#L15

This means every user can query the surcharges regardless the permission they have.

Possible Solution

An example of a query that has the desired permission validation. https://github.com/reactioncommerce/reaction/blob/b11e47f05a3d3042b76385e992960dfafb36a286/packages/api-plugin-accounts/src/queries/groups.js#L14

Guia do colaborador

Direção de pesquisa
Compare a consulta surcharges com a consulta groups em api plugin accounts para entender o padrão de validação de permissão. Implemente a verificação de permissão `read` ausente na consulta surcharges.
Pilha de tecnologia
javascriptnodejsgraphql
Domain
backendsecurity
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Node.jsGraphQL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70