Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

merge the id with the rest of the options sent to Sso #27

Merged
merged 1 commit into from

2 participants

@carlhoerberg

The problem that this patch fixes is that if you type kensa sso 123 the ID won't be added to the post data sent to the server

I guess there should be some kind of test for this part but i couldn't find any nor a good point to add new ones.. =/

@carlhoerberg

oh, dup of #26

@csquared csquared reopened this
@csquared csquared merged commit 49cd885 into from
@csquared

kensa v1.4.1 just released with this change!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/heroku/kensa/client.rb
View
2  lib/heroku/kensa/client.rb
@@ -78,7 +78,7 @@ def run
def sso
id = @args.shift || abort("! no id specified; see usage")
data = decoded_manifest
- sso = Sso.new(data.merge(@options)).start
+ sso = Sso.new(data.merge(@options).merge(:id => id)).start
puts sso.message
Launchy.open sso.sso_url
end
Something went wrong with that request. Please try again.