varvet/pundit

Overriding both authorize and policy is required for namespacing

Aperta

#723 aperta il 20 feb 2022

Vedi su GitHubGuida contributor
 (4 commenti) (1 reazione) (0 assegnatari)Ruby (643 fork)batch import
good first issue

Metriche repository

Star
 (8501 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The README explains how to namespace policies, however this does not seem to take into account direct calls to policy (e.g. calling policy(record).show? from some view). Before the merge of #697, it was possible to only override policy and authorize would automatically be properly scoped as well. Now it has apparently become necessary to override both authorize and policy in Pundit::Authorization. It still works, but is just now more cumbersome, and also required some figuring out when performing the upgrade to Pundit 2.2.0.

Opening this issue as suggested in #697.

Guida contributor

Direzione di ricerca
Ispeziona la PR #697 e traccia come i metodi `authorize` e `policy` vengono sovrascritti. Identifica la delega mancante nella logica dei namespace e proponi una correzione che ripristini lo scoping automatico.
Tech stack
ruby
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Ruby
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50