FlowKit icon indicating copy to clipboard operation
FlowKit copied to clipboard

Scope checking should be sets, not lists

Open Thingus opened this issue 1 year ago • 0 comments

From https://github.com/Flowminder/FlowKit/pull/6581#discussion_r1595142083

It's compared to other lists when we check it against the requested scopes; now I say that, a set would be better there too, but that probably belongs in a different PR

Given that there's no reason we need duplicate scopes, it's probably sensible to be passing scopes/permissions around as sets instead of lists

Thingus avatar May 09 '24 10:05 Thingus