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(concourse): support old and new concourse auth (#762) #823

Conversation

dogonthehorizon
Copy link
Member

@dogonthehorizon dogonthehorizon commented Jul 21, 2020

  • fix(concourse): support old and new concourse auth

Fixes concourse authentication for clusters >= v6.1.0 by detecting version and providing different implementation accordingly.

Addresses issue: spinnaker/spinnaker#5797

  • fix(concourse): support old and new concourse auth

Sending basic auth header to /info endpoint causes 401

Addresses issue: spinnaker/spinnaker#5797

Co-authored-by: Jared Stehler jared.stehler@edgenuity.com

* fix(concourse): support old and new concourse auth

Fixes concourse authentication for clusters >= v6.1.0 by detecting version and providing different implementation accordingly.

Addresses issue: spinnaker/spinnaker#5797

* fix(concourse): support old and new concourse auth

Sending basic auth header to /info endpoint causes 401

Addresses issue: spinnaker/spinnaker#5797

Co-authored-by: Jared Stehler <jared.stehler@edgenuity.com>
@dogonthehorizon
Copy link
Member Author

dogonthehorizon commented Jul 21, 2020

FYI @jaredstehler @kevinawoo manually cherry-picked this because mergify seems to be having issues

@spinnakerbot
Copy link
Contributor

Please delete the pull request instructions from the body of your pull request message.

The instructions start with the line:

We prefer small, well tested pull requests.

You can reopen your pull request when this has been addressed.

@kevinawoo kevinawoo added the ready to merge Approved and ready for merge label Jul 21, 2020
@kevinawoo kevinawoo reopened this Jul 21, 2020
@mergify mergify bot added the auto merged label Jul 21, 2020
@ezimanyi
Copy link
Contributor

Fixed the branch protection to no longer expect a Travis build to pass and instead to only expect the Github Action build to pass.

@ezimanyi
Copy link
Contributor

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Jul 22, 2020

Command refresh: success

@mergify mergify bot merged commit 4fe1326 into spinnaker:release-1.21.x Jul 22, 2020
@kevinawoo kevinawoo deleted the ff/mergify/bp/release-1.21.x/pr-762 branch July 22, 2020 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants