-
Notifications
You must be signed in to change notification settings - Fork 41
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
API 1.1 ongoing review PR #258
Conversation
788a327
to
083ff45
Compare
We have seen authUrl retrieving HTML files greater than 500kb in size which should have been rejected due to the content type, but also because of the size.
* spec: Clarify EventEmitter emit behaviour when new listeners registered * spec: Update to reflect Simon’s suggested wording
083ff45
to
7fbc62c
Compare
Add missing token and tokenDetails properties to authOptions Clarify TokenDetails
In order to improve libraries for all customers, we are proposing that all libraries send unhandled exceptions to our central error reporting service.
cc0bb4b
to
2d11e3e
Compare
Clarify spec on client auth failures
Fixes #338
nanoc buildpack has default https://github.com/ably-forks/heroku-buildpack-nanoc
See commits a49056e...79ffa18
Yes |
Done, in https://github.com/ably/docs/tree/integration-spec-1-1
In progress
I'm proposing to defer (ably/specification#71)
Yes
Done
Yes
Done already |
Fix RSL1g duplicate
Spec should describe emitting ConnectionEvent not ConnectionState
I've now combined master-1-1 and integration-spec-1-1 and promoted the result to master. The previous master is now https://github.com/ably/docs/tree/v1.0-as-amended |
We will use this PR to track ongoing proposed changes for the 1.1 specification. Commits merged in are generally accepted to be included in 1.1