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 App SSH (Broken when auth and token endpoints are different) #2250

Merged
merged 1 commit into from
Jun 4, 2018

Conversation

nwmac
Copy link
Contributor

@nwmac nwmac commented May 31, 2018

We are incorrectly using the Auth endpoint and not the token endpoint - for many providers these are the same, but if they are not, SSH is broken.

Also improved the logging

@cfdreddbot
Copy link

Hey nwmac!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@nwmac nwmac changed the title Fix App SSH Fix App SSH (Broken when auth and token endpoints are different) May 31, 2018
@codecov
Copy link

codecov bot commented May 31, 2018

Codecov Report

Merging #2250 into v2-master will decrease coverage by 0.32%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           v2-master    #2250      +/-   ##
=============================================
- Coverage      71.78%   71.46%   -0.33%     
=============================================
  Files            565      576      +11     
  Lines          23192    24292    +1100     
  Branches        5205     5443     +238     
=============================================
+ Hits           16649    17360     +711     
- Misses          6543     6932     +389

@irfanhabib
Copy link
Contributor

LGTM

@irfanhabib irfanhabib merged commit e98f2f3 into v2-master Jun 4, 2018
@irfanhabib irfanhabib deleted the fix-ssh branch June 4, 2018 09:49
@andrewghobrial
Copy link
Contributor

Thanks for adding this!

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

Successfully merging this pull request may close these issues.

4 participants