Skip to content

Register OAuth client as "Posit CLI" - #871

Merged
nealrichardson merged 1 commit into
mainfrom
npr-870
Oct 16, 2026
Merged

nealrichardson merged 1 commit into
mainfrom
npr-870

Conversation

@nealrichardson

@nealrichardson nealrichardson commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #870.

Connect now shows the OAuth client name on its consent screen, so register with the human-readable name Posit CLI instead of rsconnect-python.

  • The keyring service name stays rsconnect-python. It's the key for stored tokens, and changing it would orphan existing logins.
  • Clients that are already registered keep their old name until they register again. The CHANGELOG notes this.
  • Added a test that checks the client_name sent during registration.
image

Connect now shows the client name on its consent screen, so use a
human-readable name instead of "rsconnect-python". The keyring service
name is unchanged so existing stored credentials keep working.

Fixes #870

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-10-16 16:23 UTC

@github-actions

Copy link
Copy Markdown

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
10750 9382 87% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
rsconnect/oauth.py 86% 🟢
TOTAL 86% 🟢

updated for commit: 9af4c55 by action🐍

@nealrichardson
nealrichardson merged commit 814c243 into main Oct 16, 2026
27 checks passed
@nealrichardson
nealrichardson deleted the npr-870 branch October 16, 2026 16:23
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.

Change oauth client name?

2 participants