Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

Fix: #2786 - Pass cookiecontainer to HttpClientHandler to support provisioning client side pages on tenant with Federated authentication login #2787

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

Conversation

oppie85
Copy link

@oppie85 oppie85 commented Oct 6, 2020

Q A
Bug fix? yes
New feature? no
New sample? no
Related issues? fixes #2786

What's in this Pull Request?

In the ClientSidePage code, serveral non-CSOM API's are called (example: GetClientSideWebPartsAsync). These are authenticated using either the explicit credentials or Authorization header. However, when dealing with a tenant that uses Federation, these calls will throw an error unless the Federation cookies are also passed to the HttpClientHandler. This fix reads the cookies in InitializeSecurityAsync and passes them to all the relevant API's in the same manner that the accessToken is.

Pass cookiecontainer to HttpClientHandler to support  provisioning client sage pages on tenant sith Federated authentication login
@oppie85 oppie85 changed the title Fix: #2786 - Pass cookiecontainer to HttpClientHandler to support provisioning client sage pages on tenant sith Federated authentication login Fix: #2786 - Pass cookiecontainer to HttpClientHandler to support provisioning client side pages on tenant with Federated authentication login Oct 6, 2020
@jansenbe jansenbe self-assigned this Oct 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants