seatgeek / api-support

A support channel for the SeatGeek Platform
9 stars 7 forks source link

[Documentation] Incorrectly states that non-SSL connections allowed #2

Closed stephenyeargin closed 8 years ago

stephenyeargin commented 8 years ago

When viewing the documentation, all of the links to API resources are written as http://api.seatgeek.com/2/<method>. However, as of a recent change, all of these are being redirect to their https counterparts.

$ curl -I http://api.seatgeek.com/2
HTTP/1.1 301 Moved Permanently
Server: openresty
Date: Tue, 09 Feb 2016 03:54:02 GMT
Content-Type: text/html
Content-Length: 178
Connection: keep-alive
Location: https://api.seatgeek.com/2
X-Served-By: lb-14-04-i-8152c308

The implication here is that certain code, such as a SeatGeek gem, does not properly handle the redirect.

Related: https://github.com/dmvt/seatgeek/issues/4

How to resolve:

josegonzalez commented 8 years ago

Good catch! We made this change a few weeks ago. I'll update the documentation.

josegonzalez commented 8 years ago

@stephenyeargin The new docs should be out soon. I totally neglected the Google Groups, and will make an announcement about this there ASAP.

Again, thanks for the heads up!