Johanna LarssonandTangled eebce28926 fix: handle wildcard + aud in OAuth scopes ...

`rpc:*?aud=did:web:api.bsky.app#bsky_appview` fails in the permissions check even though it's a valid scope. This makes it resolve correctly.

Also this commit fixes a bug where `rpc` as a bare scope is interpreted as all methods for all services, but it's really an invalid scope. Can drop it if it's a problem, but it just looks like a bug in the implementation rather than something intended?

Note that the parser treats `rpc:*` as "allow everything for no services", but aud is actually required, so it's not a valid scope? Didn't mess with it though because delegation uses it and that part of the code treats it as allow everything for every service. We should maybe split the scope parsers so we have one for delegated scopes and one for regular?

However, to avoid the same bug as bare `rpc`, I changed the enforcement step in "regular" OAuth to disallow `rpc:*` (this matches the existing behavior on main, no actual change, just a little acknowledgement).

2026-09-23 15:42:29 +00:00

..

src

fix: handle wildcard + aud in OAuth scopes

2026-09-23 15:42:29 +00:00

Cargo.toml

types: sqlx finally behind a feature -> derive Ord & Borrow on newtypes

2026-07-11 08:30:22 +03:00