You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
When will support for new Personal Access Tokens instead of API Keys begin? #61
According to the Intercom docs, API Keys are deprecated and will be fully replaced with a Personal Access Token in January 2017. I was expecting NewClient to support this change already ie.
ic := intercom.NewClient("appID", "apiKey") would be ic := intercom.NewClient("pat")
When can we expect the package to update to support this change?
Hi @briemarie - this should work right now by passing the PAT where you would normally pass the appID, and send an empty string where the API key would normally be.
The API documentation for Personal Access Tokens is explicit about using the Bearer authorization type. It looks like the solution mentioned above works since the Basic authorization type is overloaded to accept a PAT where username usually goes.
@josler I can't find any documentation for the overloading. Is that a stable API feature I can depend on?
Just to note that we did update the README here to clarify how to use access tokens to authorize access
ic := intercom.NewClient("access_token", "")
But we will create bearer access for this once we get a chance so will leave it open for now
Activity
josler commentedon Dec 9, 2016
Hi @briemarie - this should work right now by passing the PAT where you would normally pass the appID, and send an empty string where the API key would normally be.
timhudson commentedon Jan 11, 2017
The API documentation for Personal Access Tokens is explicit about using the
Bearerauthorization type. It looks like the solution mentioned above works since theBasicauthorization type is overloaded to accept a PAT whereusernameusually goes.@josler I can't find any documentation for the overloading. Is that a stable API feature I can depend on?
kant01ne commentedon Jan 11, 2017
Hey @timhudson
The API documentation recommend to use bearer tokens but basic auth should work too so you can safely use @josler's solution above.
choran commentedon Aug 21, 2017
Just to note that we did update the README here to clarify how to use access tokens to authorize access
ic := intercom.NewClient("access_token", "")
But we will create bearer access for this once we get a chance so will leave it open for now
zwass commentedon Apr 12, 2018
This also needs to be updated in the GoDoc (https://godoc.org/gopkg.in/intercom/intercom-go.v2#NewClient). It just misled me and resulted in a fair bit of time debugging.