Skip to content

Conversation

@hf
Copy link
Contributor

@hf hf commented Apr 25, 2025

As unfortunately documented almost nowhere, sending a request to the Microsoft Graph API requires a query parameter to specify an undocumented api-version with value 1.6.

Source: https://stackoverflow.com/questions/51085863/retrieve-group-claims-using-claim-sources-returns-the-specified-api-version-is

@hf hf requested a review from a team as a code owner April 25, 2025 11:37
@hf hf force-pushed the hf/azure-overage-include-api-version branch from 89f3b9e to 3a20eb7 Compare April 25, 2025 11:49
@hf
Copy link
Contributor Author

hf commented Apr 25, 2025

Merging with admin to unblock a customer.

@hf hf merged commit 44890d0 into master Apr 25, 2025
3 checks passed
@hf hf deleted the hf/azure-overage-include-api-version branch April 25, 2025 11:57
@coveralls
Copy link

Pull Request Test Coverage Report for Build 14663886363

Details

  • 11 of 11 (100.0%) changed or added relevant lines in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.007%) to 69.448%

Files with Coverage Reduction New Missed Lines %
internal/api/provider/azure.go 2 80.72%
Totals Coverage Status
Change from base Build 14663286023: 0.007%
Covered Lines: 10886
Relevant Lines: 15675

💛 - Coveralls

hf pushed a commit that referenced this pull request May 5, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.172.0](v2.171.0...v2.172.0)
(2025-05-04)


### Features

* fix large group claim handling in azure id tokens
([#1995](#1995))
([2f323fe](2f323fe))
* use `global_user_id` over `sub` for `vercel_marketplace` issuer
([#1990](#1990))
([f94f97e](f94f97e))


### Bug Fixes

* azure overage claims start with single `_` not two
([#1999](#1999))
([29f3440](29f3440))
* remove azure claim overage code.
([#2005](#2005))
([63dce14](63dce14))
* resolving azure overage claim should include `api-version=1.6` query
parameter ([#2000](#2000))
([44890d0](44890d0))
* upgrade godotenv to v1.5.1 to fix multiline file loading
([#1997](#1997))
([f2af4b2](f2af4b2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
wdoppenberg pushed a commit to wdoppenberg/supabase-auth that referenced this pull request Jun 5, 2025
…uery parameter (supabase#2000)

As unfortunately documented almost nowhere, sending a request to the
Microsoft Graph API requires a query parameter to specify an
undocumented `api-version` with value `1.6`.

Source:
https://stackoverflow.com/questions/51085863/retrieve-group-claims-using-claim-sources-returns-the-specified-api-version-is
cemalkilic pushed a commit that referenced this pull request Aug 7, 2025
…uery parameter (#2000)

As unfortunately documented almost nowhere, sending a request to the
Microsoft Graph API requires a query parameter to specify an
undocumented `api-version` with value `1.6`.

Source:
https://stackoverflow.com/questions/51085863/retrieve-group-claims-using-claim-sources-returns-the-specified-api-version-is
cemalkilic pushed a commit that referenced this pull request Aug 7, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.172.0](v2.171.0...v2.172.0)
(2025-05-04)


### Features

* fix large group claim handling in azure id tokens
([#1995](#1995))
([2f323fe](2f323fe))
* use `global_user_id` over `sub` for `vercel_marketplace` issuer
([#1990](#1990))
([f94f97e](f94f97e))


### Bug Fixes

* azure overage claims start with single `_` not two
([#1999](#1999))
([29f3440](29f3440))
* remove azure claim overage code.
([#2005](#2005))
([63dce14](63dce14))
* resolving azure overage claim should include `api-version=1.6` query
parameter ([#2000](#2000))
([44890d0](44890d0))
* upgrade godotenv to v1.5.1 to fix multiline file loading
([#1997](#1997))
([f2af4b2](f2af4b2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
issuedat pushed a commit that referenced this pull request Sep 30, 2025
…uery parameter (#2000)

As unfortunately documented almost nowhere, sending a request to the
Microsoft Graph API requires a query parameter to specify an
undocumented `api-version` with value `1.6`.

Source:
https://stackoverflow.com/questions/51085863/retrieve-group-claims-using-claim-sources-returns-the-specified-api-version-is
issuedat pushed a commit that referenced this pull request Sep 30, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.172.0](v2.171.0...v2.172.0)
(2025-05-04)


### Features

* fix large group claim handling in azure id tokens
([#1995](#1995))
([2f323fe](2f323fe))
* use `global_user_id` over `sub` for `vercel_marketplace` issuer
([#1990](#1990))
([f94f97e](f94f97e))


### Bug Fixes

* azure overage claims start with single `_` not two
([#1999](#1999))
([29f3440](29f3440))
* remove azure claim overage code.
([#2005](#2005))
([63dce14](63dce14))
* resolving azure overage claim should include `api-version=1.6` query
parameter ([#2000](#2000))
([44890d0](44890d0))
* upgrade godotenv to v1.5.1 to fix multiline file loading
([#1997](#1997))
([f2af4b2](f2af4b2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants