fix: meet scope mapping and reply QP soft-wrap corruption#6
Closed
shigechika wants to merge 1 commit intomainfrom
Closed
fix: meet scope mapping and reply QP soft-wrap corruption#6shigechika wants to merge 1 commit intomainfrom
shigechika wants to merge 1 commit intomainfrom
Conversation
- fix(auth): map meet service to meetings scope prefix so that `gws auth login -s meet` correctly requests all Meet scopes. Mirrors upstream PR googleworkspace#754. - fix(gmail): pre-wrap quoted body to 73 chars in +reply plain-text mode so that lines never exceed 75 chars after the "> " prefix, preventing quoted-printable soft-wrap from injecting spurious "> " mid-sentence. Fixes upstream issue googleworkspace#769.
Owner
Author
|
Splitting into separate PRs. reply.rs QP wrap fix is dropped — non-ASCII byte-length handling is a rabbit hole best left for a dedicated fix. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
gws auth login -s meetが Meet スコープを1件も要求しない問題を修正。map_service_to_scope_prefixesに"meet" => vec!["meetings"]を追加。upstream PR fix(auth): map meet service to meetings scope prefix googleworkspace/cli#754 に相当。+replyプレーンテキストモードで、元メールの本文が70〜75文字付近でハードラップされていると、>プレフィックス付加後に76文字を超えた行が Quoted-Printable のソフトラップ (=\r\n) を引き起こし、引用テキスト中に>が挿入されて文章が崩れるバグを修正。format_quoted_originalでプレフィックス付加前に73文字でプリラップするよう修正。upstream issue +reply (plain text) corrupts quoted parent when hard-wrapped lines push prefixed lines past 76 chars googleworkspace/cli#769 に相当。Test plan
cargo clippy -- -D warnings— 警告なしcargo test— 826テスト全通過scope_matches_service_meet(auth)test_format_quoted_original_long_lines_do_not_exceed_75_chars,test_word_wrap_preserves_short_lines,test_word_wrap_breaks_long_line(reply)References
🤖 Generated with Claude Code