Skip to content

Support FIC with AT_POP - #3299

Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom
sruthi/AtPopWithJwk
Mar 24, 2025
Merged

Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom
sruthi/AtPopWithJwk

Conversation

@sruke

@srukesruthikeerthi (sruke) commented Mar 14, 2025

Copy link
Copy Markdown
Contributor

No description provided.

jmprieur
Jean-Marc Prieur (jmprieur) requested changes Mar 14, 2025
View reviewed changes

@jmprieurJean-Marc Prieur (jmprieur) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes
(or I miss something)

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs Outdated
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs Outdated
@srukesruthikeerthi (sruke) changed the title Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke) commented Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke) marked this pull request as ready for review March 23, 2025 15:28
@sruke
sruthikeerthi (sruke) requested a review from a team as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur) approved these changes Mar 24, 2025
View reviewed changes

@jmprieurJean-Marc Prieur (jmprieur) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks sruthikeerthi (@sruke)

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19 reviewed Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19 reviewed Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19 approved these changes Mar 24, 2025
View reviewed changes

@jennyf19jennyf19 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:
approve w/small suggestions

Sruthi Keerthi Rangavajhula (from Dev Box)and others added 6 commits March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke) linked an issue Mar 24, 2025 that may be closed by this pull request
[Feature Request] Support Federated Identity Credentials (FIC) with AtPop #3309
Closed
@sruke
sruthikeerthi (sruke) merged commit adfa178 into masterMar 24, 2025
@sruke
sruthikeerthi (sruke) deleted the sruthi/AtPopWithJwk branch March 24, 2025 21:51
This was referenced Jul 25, 2025
Merged
Merged
Merged
Closed
Closed
@dependabotdependabotBot mentioned this pull request Aug 1, 2025
Closed
This was referenced Apr 27, 2026
Open
Closed
Open
Merged
Merged
Merged
Closed
Closed
Closed
This was referenced May 4, 2026
Open
Closed
Open
Open
Closed
Open
Merged
Merged
Closed
This was referenced May 25, 2026
Merged
Closed
@dependabotdependabotBot mentioned this pull request Jun 3, 2026
Open
This was referenced Jun 19, 2026
Merged
Closed
This was referenced Jul 2, 2026
Closed
Closed
Closed
This was referenced Jul 13, 2026
Closed
Closed
Open
@dependabotdependabotBot mentioned this pull request Aug 19, 2026
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop

6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
Support FIC with AT_POP by sruke · Pull Request #3299 · AzureAD/microsoft-identity-web · GitHub
Skip to content
Support FIC with AT_POP - #3299
Merged
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Mar 24, 2025
Merged
Support FIC with AT_POP#3299
sruthikeerthi (sruke) merged 10 commits into
masterfrom 
sruthi/AtPopWithJwk
Conversation
@sruke
@srukesruthikeerthi (sruke)
commented
Mar 14, 2025

Copy link


Copy Markdown

Contributor
 No description provided. 
jmprieur
Jean-Marc Prieur (jmprieur)
requested changes
Mar 14, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that these are the right changes

(or I miss something)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
Outdated

Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
Outdated

@srukesruthikeerthi (sruke)
changed the title
Support providing a custom signed assertion with AT_POPSupport FIC with AT_POPMar 21, 2025
sruke
sruthikeerthi (sruke)
commented
Mar 23, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
@sruke
sruthikeerthi (sruke)
marked this pull request as ready for review
March 23, 2025 15:28
@sruke
sruthikeerthi (sruke)
requested a review
from a team
as a code ownerMarch 23, 2025 15:28
jmprieur
Jean-Marc Prieur (jmprieur)
approved these changes
Mar 24, 2025
View reviewed changes
@jmprieurJean-Marc Prieur (jmprieur)
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks sruthikeerthi (@sruke)
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/MsAuth10AtPop.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
reviewed
Mar 24, 2025
View reviewed changes
Comment threadsrc/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
jennyf19
jennyf19
approved these changes
Mar 24, 2025
View reviewed changes
@jennyf19jennyf19
left a comment


Copy link


Copy Markdown

Collaborator
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

approve w/small suggestions
Sruthi Keerthi Rangavajhula (from Dev Box)and others
added 6 commits
March 24, 2025 14:10
Sruthi Keerthi Rangavajhula (from Dev Box)
Adds AtPopOperationTests
d380e08
@ksaafKashif MehmoodSruthi Keerthi Rangavajhula (from Dev Box)
Make Microsoft.Extensions.Http dependency framework friendly (#3296)
d1b0021
This PR pins Microsoft.Extensions.Http ver 3.1.3 to .Net Framework/Standard and uses inbox version of Microsoft.Extensions.Http for .Net Core.
Change involves updating Directory.Build.props and updating CSProj to reference Microsoft.Extensions.Http package only for non .Net Core.
---------
Co-authored-by: Kashif Mehmood <kashifm@ntdev.microsoft.com>
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
Update to IdentityModel 8.7.0 (#3307)
3438224
@pmaytakSruthi Keerthi Rangavajhula (from Dev Box)
3.8.1 changelog (#3306)
76a36a0
@trwalkeSruthi Keerthi Rangavajhula (from Dev Box)
Update Directory.Build.props (#3305)
2d319cb
@sruke@jennyf19Sruthi Keerthi Rangavajhula (from Dev Box)
Update src/Microsoft.Identity.Web.TokenAcquisition/AtPopOperation.cs
d7e29d3
Co-authored-by: jennyf19 <jeferrie@microsoft.com>
@srukesruthikeerthi (sruke)
linked an issue
Mar 24, 2025
that may be
closed
by this pull request


[Feature Request] Support Federated Identity Credentials (FIC) with AtPop
#3309

Closed

@sruke
sruthikeerthi (sruke)
merged commit adfa178
into
masterMar 24, 2025
@sruke
sruthikeerthi (sruke)
deleted the

sruthi/AtPopWithJwk

branch
March 24, 2025 21:51

This was referenced Jul 25, 2025
 Merged

 Merged

 Merged

 Closed

 Closed

@dependabotdependabotBot
mentioned this pull request
Aug 1, 2025
 Closed


This was referenced Apr 27, 2026
 Open

 Closed

 Open

 Merged

 Merged

 Merged

 Closed

 Closed

 Closed


This was referenced May 4, 2026
 Open

 Closed

 Open

 Open

 Closed

 Open

 Merged

 Merged

 Closed


This was referenced May 25, 2026
 Merged

 Closed

@dependabotdependabotBot
mentioned this pull request
Jun 3, 2026
 Open


This was referenced Jun 19, 2026
 Merged

 Closed


This was referenced Jul 2, 2026
 Closed

 Closed

 Closed


This was referenced Jul 13, 2026
 Closed

 Closed

 Open

@dependabotdependabotBot
mentioned this pull request
Aug 19, 2026
 Closed

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

Reviewers

@jmprieurJean-Marc Prieur (jmprieur)jmprieur approved these changes
@jennyf19jennyf19jennyf19 approved these changes

Assignees

@srukesruthikeerthi (sruke)

Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

[Feature Request] Support Federated Identity Credentials (FIC) with AtPop


6 participants

@sruke@jmprieur@jennyf19@ksaaf@pmaytak@trwalke