-
Notifications
You must be signed in to change notification settings - Fork 86
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
Access to sni-endpoints API #49
Comments
I've forked the code and re-ran the build against the beta schema: https://github.com/jalada/platform-api |
Historically we have left out non-GA stuff from the gem (as it isn't in the core schema yet). I wouldn't go so far to say that this was a policy, so much as coincidence of tooling. I don't think we could really automate this reasonably, since some variants actually change behavior of existing endpoints it would be weird trying to merge them. That said, this may be reflective of it just being about time for us to bring this out of variant and into the main schema. Does that help/make sense? |
@geemus Seems fair. Yep, it makes sense. I'm also an idiot because I didn't realise you could pass custom headers using this gem. Thanks! |
@maxehmookau hey, there are many parts of this gem that are (unfortunately) less than obvious right now. Definitely appreciate your understanding and definitely let us know (or PR) if there are some good fixes to docs or other things to improve the clarity. Thanks! |
If I understand correctly: As long as the new features "http-sni" and "sni-endpoints" are in beta, they won't be implemented in the api? So to use the new features I would do something like Thanks! |
@chriso0710, https://github.com/heroku/platform-api#passing-custom-headers |
@maxehmookau yes, but that's not enough to access endpoints that aren't in the schema generated when packaging up this gem, because the method isn't defined, right? |
Ah yeah, you are right. Then |
Thanks, guys! IMHO the new SNI/SSL feature is really huge and could be worth implementing it as beta into the api. |
For now, yes, you need in some capacity to explicitly set the headers. The method described should work, as would other http-libraries used directly (I'm partial to excon/excon), but unfortunately it lacks some of the convenience and syntactic sugar provided by platform-api. |
Now that SNI-based SSL is out of beta will this be supported? |
Yes. You can find it here: https://devcenter.heroku.com/articles/platform-api-reference#sni-endpoint |
What is the status of implementation of sni endpoints in this gem? |
+1 |
Take a look at This issue should be reopened IMHO. |
@chriso0710 good call. There are a couple open pull requests relating to regenerating schema and releasing also. Lots of stuff going on, but hoping to get to it pretty soon. |
I would really like to stop telling people to use my fork, as it looks like it might be causing further issues as it falls behind now 😞 pixielabs/letsencrypt-rails-heroku#35 |
@mathias perhaps you can speak to where we are in this process? |
Is this going to be fixed? And why is this ticket closed? This is still an open issue. |
It will be fixed, though I'm not sure of exact timing. I'm not actually certain why it was closed (that was some time ago), I've re-opened. |
This appears to be related to pull request 56. Any timeline? Soon? We can't use the new Heroku support for Let's Encrypt, because we use a CDN - so this functionality is important to us. |
Hello, These methods should be present in the 1.0.1 release of this gem. # in pry:
require 'platform-api'
#=> true
client = PlatformAPI.connect_oauth(ENV['TOKEN'])
#=> #<PlatformAPI::Client:... @client=#<Heroics::Client url="https://api.heroku.com">>
ls -m client.sni_endpoint
#=> PlatformAPI::SniEndpoint#methods: create delete info list update
client.sni_endpoint.list('young-garden-2012')
#=> [] Hope this helps! I'm going to close this for now. Feel free to comment+re-open if something is amiss. |
The new sni-endpoints API appears to be available (albeit in beta) and is being used by the production dashboard at heroku.com.
Is there a general policy against including beta endpoints in the ruby gem? If not, I'll happily put together a PR to add this functionality.
The text was updated successfully, but these errors were encountered: