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

Add authorization headers to Paket Push #2034

Merged
merged 2 commits into from Nov 22, 2016

Conversation

Projects
None yet
2 participants
@varon
Contributor

varon commented Nov 22, 2016

This adds authorization headers from the config, if they're present, to the client.
This fixes #2033, and allows Paket.Push to work with VSTS.

Better verbose logging was also added.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 22, 2016

Member

awesome!

Member

forki commented Nov 22, 2016

awesome!

@forki forki merged commit 09ed8d5 into fsprojects:master Nov 22, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
@varon

This comment has been minimized.

Show comment
Hide comment
@varon

varon Nov 22, 2016

Contributor

The CI build seemed to fail. Was that related to my code, or is it a separate issue?

I've tested this on my side and had no trouble compiling.

Contributor

varon commented Nov 22, 2016

The CI build seemed to fail. Was that related to my code, or is it a separate issue?

I've tested this on my side and had no trouble compiling.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 22, 2016

Member

CI build is currently broken. but your fix is released. thanks a lot

Member

forki commented Nov 22, 2016

CI build is currently broken. but your fix is released. thanks a lot

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