Skip to content
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

Improve connection API (where most user mistakes occur) #78

Closed
alanseiden opened this issue May 10, 2016 · 1 comment
Closed

Improve connection API (where most user mistakes occur) #78

alanseiden opened this issue May 10, 2016 · 1 comment

Comments

@alanseiden
Copy link
Collaborator

alanseiden commented May 10, 2016

Reduce incidence of customer mistakes with a clearer API to connect to the toolkit.

One solution: allow array of options to be passed to getInstance() and constructor (either as param or with fluent interface). This would let us provide examples of a connection that does all necessary to define the connection. Solve several problems, including:
◦ users forgetting to add a separate 'stateless'/'InternalKey' setOptions statement, causing various nightmares
◦ inability to specify in code that we want to use HTTP transport (currently we have to specify in toolkit.ini with URL); we need to know the connection type and URL at connection time, not later. (Alan, clarify this issue)

@alanseiden alanseiden changed the title Improve connection code Improve connection API Jun 28, 2016
@alanseiden alanseiden changed the title Improve connection API Improve connection API (where most user mistakes occur) Jun 28, 2016
@alanseiden
Copy link
Collaborator Author

We will address this issue cleanly in toolkit 2.0. See #87.
Meanwhile, let us help people by creating more examples that illustrate proper connection techniques. See #71.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant