Skip to content

Add the "images" scope to OAuthScopes - #700

Merged
zliang-akamai merged 2 commits into
linode:devfrom
borntyping:add-images-oauth-scope
Aug 24, 2026
Merged

Add the "images" scope to OAuthScopes#700
zliang-akamai merged 2 commits into
linode:devfrom
borntyping:add-images-oauth-scope

Conversation

@borntyping

Copy link
Copy Markdown
Contributor

📝 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
@zliang-akamai
zliang-akamai merged commit 301fad1 into linode:dev Aug 24, 2026
17 of 18 checks passed
@yec-akamai yec-akamai added the improvement for improvements in existing functionality in the changelog. label Sep 10, 2026
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. improvement for improvements in existing functionality in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants