Skip to content

Add the "images" scope to OAuthScopes - #700

Open
borntyping wants to merge 2 commits into
linode:devfrom
borntyping:add-images-oauth-scope
Open

Add the "images" scope to OAuthScopes#700
borntyping wants to merge 2 commits into
linode:devfrom
borntyping:add-images-oauth-scope

Conversation

@borntyping

Copy link
Copy Markdown

📝 Description

It's not currently possible to create a LinodeLoginClient that supports the images:read_write or images:read_only scope.

✔️ How to Test

I tested this by duplicating the new Images class in my own application and passing it to LinodeLoginClient.generate_login_url, then testing that my application could list images after authenticating.

@borntyping
borntyping requested review from a team as code owners May 21, 2026 14:08
@borntyping
borntyping requested review from lgarber-akamai and mawilk90 and removed request for a team May 21, 2026 14:08
@zliang-akamai
zliang-akamai requested a balanced review from Copilot August 18, 2026 19:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Images OAuth scope support to LinodeLoginClient.

Changes:

  • Adds OAuthScopes.Images access levels.
  • Registers Images for parsing and wildcard scopes.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread linode_api4/login_client.py
@zliang-akamai zliang-akamai added the community-contribution contributions from the community. label Aug 18, 2026
@zliang-akamai
zliang-akamai requested review from a team, jriddle-linode, mgwoj and psnoch-akamai and removed request for a team August 18, 2026 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution contributions from the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants