Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(azure): handle credentials scopes for Graph #7239

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pedrooot
Copy link
Member

@pedrooot pedrooot commented Mar 14, 2025

Context

Fix #7214

Description

This PR handles management scopes for ms-graph.

Work done with my colleague @puchy22 🚀

Checklist

API

  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, Poetry, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@pedrooot pedrooot requested review from a team as code owners March 14, 2025 10:18
@github-actions github-actions bot added the provider/azure Issues/PRs related with the Azure provider label Mar 14, 2025
@pedrooot pedrooot added the no-merge Please, DO NOT MERGE this PR. label Mar 14, 2025
Copy link

codecov bot commented Mar 14, 2025

Codecov Report

Attention: Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 88.64%. Comparing base (f2e19d3) to head (a8211e9).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7239      +/-   ##
==========================================
- Coverage   92.23%   88.64%   -3.60%     
==========================================
  Files          81     1219    +1138     
  Lines        7356    35554   +28198     
==========================================
+ Hits         6785    31517   +24732     
- Misses        571     4037    +3466     
Flag Coverage Δ
api ?
prowler 88.64% <80.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
prowler 88.64% <80.00%> (∅)
api ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-merge Please, DO NOT MERGE this PR. provider/azure Issues/PRs related with the Azure provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Running into 302 error Subscription could not be found
1 participant