Skip to content

Bump oauthlib from 3.3.1 to 4.0.0 - #282

Closed
Radhika Gupta (rads-1996) wants to merge 4 commits into
microsoft:mainfrom
rads-1996:update-oauth
Closed

Radhika Gupta (rads-1996) wants to merge 4 commits into
microsoft:mainfrom
rads-1996:update-oauth

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 24,194.3 24,454.7 -1.06% ✅
azure_monitor_span yes 160,025.6 160,025.6 +0.00% ✅
otel_log no 28,189.7 27,954.0 +0.84% ✅
otel_span no 48,730.6 48,236.9 +1.02% ✅

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.

Copilot review overview

🟢 Approval recommended

The dependency metadata is consistent, compatible with supported Python versions, and the unrelated test edit is nonfunctional.

Review effort: Balanced
Findings: None

What changed in this PR

Updates the locked OAuthlib dependency to 4.0.0.

Changes:

  • Updates OAuthlib version and artifacts.
  • Applies a formatting-only test change.
File Description
uv.lock Locks OAuthlib 4.0.0 and hashes.
tests/​a365/​test_persistent_storage.py Reformats a SQL execution call.

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

Copilot AI balanced review requested due to automatic review settings October 2, 2026 00:01
@rads-1996 Radhika Gupta (rads-1996) added the skip-changelog Not required for changes to test files, github actions, etc. label Oct 2, 2026

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.

Copilot review overview

🔵 Needs a closer look

The major-version OAuth dependency upgrade warrants human compatibility validation.

Review effort: Balanced
Findings: None

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Not required for changes to test files, github actions, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants