Skip to content

AIP-1730: security: require oauthlib>=4.0.1 (AIP-1730) - #55

Merged
diskun00 merged 1 commit into
mainfrom
AIP-1730_oauthlib_4.0.1
Nov 10, 2026
Merged

diskun00 merged 1 commit into
mainfrom
AIP-1730_oauthlib_4.0.1

Conversation

@diskun00

@diskun00 diskun00 commented Nov 9, 2026 •

Copy link
Copy Markdown
Member

Fixes Dependabot alert #40 (oauthlib, HIGH: bearer token retrievable via device workflow with a valid client ID).

  • Jira: AIP-1730
  • oauthlib floor: >=4.0.0 → >=4.0.1 (patched version). No lockfile in this repo; Python range unchanged (>=3.10).
  • Local: resolves with databricks-cli; pytest 6 passed, 2 skipped, 4 Spark tests need a local Java runtime (CI has one).

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@diskun00
diskun00 marked this pull request as ready for review November 9, 2026 10:09
@diskun00
diskun00 requested a review from a team as a code owner November 9, 2026 10:09
@gyg-pr-tool gyg-pr-tool Bot changed the title security: require oauthlib>=4.0.1 (AIP-1730) AIP-1730: security: require oauthlib>=4.0.1 (AIP-1730) Nov 9, 2026
@coderabbitai

coderabbitai Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Summary

Risk Score: Low - Auto-merge Candidate

Raises the minimum oauthlib version from >=4.0.0 to >=4.0.1 in pyproject.toml. This patch upgrade addresses the reported security alert and reduces the risk of a bearer token being retrieved through the device workflow. No other files change.

The project’s minimum oauthlib dependency version changes from 4.0.0 to 4.0.1.

Assessment against linked issues:

Objective Addressed Explanation
Update the vulnerable oauthlib dependency AIP-1730 ✅

Suggested labels: coderabbit-risk:low

Change: Other · Severity of issue fixed: High

Merge Risk: Minimal · up to 845e9

The dependency update addresses the reported advisory. No merge-blocking issue was identified.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central TS config (base), Organization UI (inherited); Enforced overrides: Organization
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b3e03324-260a-4256-b38d-b299ff5cbf22
Commits

Reviewing files that changed from the base of the PR and between cf72966 and 845e904.

Files selected for processing: 1

  • pyproject.toml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@gyg-pr-tool
gyg-pr-tool Bot requested a review from steven-mi November 9, 2026 10:10
@diskun00
diskun00 merged commit acdbd6a into main Nov 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant