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 configured global headers #1541

Merged
merged 2 commits into from
Feb 1, 2021
Merged

fix configured global headers #1541

merged 2 commits into from
Feb 1, 2021

Conversation

tavikukko
Copy link
Contributor

Category

  • Bug fix?
  • New feature?
  • New sample?
  • Documentation update?

Related Issues

fixes #1540

What's in this Pull Request?

This fixes issue that configured global headers never got attached to actual call

@juliemturner
Copy link
Collaborator

Thanks @tavikukko -- we'll review as soon as we can.

@tavikukko
Copy link
Contributor Author

Yes, I think this is quite critical.

@patrick-rodgers
Copy link
Member

Indeed, will have a look and pushing out a release ASAP. With so many changes internally I knew we missed something, so a quick follow up release was planned. Thanks for having a look and dropping us a PR!

@patrick-rodgers patrick-rodgers merged commit ad9b66b into pnp:version-2 Feb 1, 2021
@patrick-rodgers patrick-rodgers added this to the 2.1.1 milestone Feb 1, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 7, 2022
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.

Client configurations not working
3 participants