Skip to content

fix(auth): map meet service to meetings scope prefix#7

Merged
shigechika merged 2 commits intomainfrom
fix/meet-scope-prefix
May 2, 2026
Merged

fix(auth): map meet service to meetings scope prefix#7
shigechika merged 2 commits intomainfrom
fix/meet-scope-prefix

Conversation

@shigechika
Copy link
Copy Markdown
Owner

Summary

gws auth login -s meet が Meet スコープを1件も要求しない問題を修正。

Meet API のスコープは meetings.* プレフィックス(例: meetings.space.readonly)だが、CLI の service alias は meet のため、map_service_to_scope_prefixes のフォールスルーで "meet" がそのままスコープ検索に使われ、マッチが0件になっていた。

"meet" => vec!["meetings"] を追加することで、既存の augment_with_dynamic_scopes パスが Discovery document からすべての Meet スコープを自動取得するようになる。

Test plan

  • cargo clippy -- -D warnings — 警告なし
  • cargo test — 823テスト全通過
  • 追加テスト: scope_matches_service_meet (meetings.space.{created,readonly,settings} の3スコープを検証)

References

  • Mirrors upstream PR #754
  • Fixes upstream issue #556

🤖 Generated with Claude Code

shigechika added 2 commits May 2, 2026 20:02
gws auth login -s meet silently requested zero Meet scopes because
the Meet API exposes scopes under the meetings.* prefix, not meet.*.
Add the mapping so the existing dynamic-augmentation path picks up
all Meet scopes from the Discovery document.

Mirrors upstream PR googleworkspace#754.
@shigechika shigechika merged commit ff247b1 into main May 2, 2026
10 checks passed
@shigechika shigechika deleted the fix/meet-scope-prefix branch May 2, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant