Skip to content

Storage connection string support. Fixes #339 #363

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

Merged
merged 4 commits into from
May 20, 2015
Merged

Storage connection string support. Fixes #339 #363

merged 4 commits into from
May 20, 2015

Conversation

hosungs
Copy link
Contributor

@hosungs hosungs commented May 19, 2015

Simple storage connection string support implementation and a unit test.

@zooba
Copy link
Member

zooba commented May 19, 2015

+1

account_key = connection_params.account_key
protocol = connection_params.protocol
host_base = connection_params.host_base_blob
dev_host = connection_params.dev_host_blob
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these should be host_base_queue and dev_host_queue.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your careful review and sorry about the foolish copy-paste bug. Fixed in the next commit.

@huguesv
Copy link
Contributor

huguesv commented May 20, 2015

Thank you!

huguesv pushed a commit that referenced this pull request May 20, 2015
@huguesv huguesv merged commit 00b8b50 into Azure:master May 20, 2015
@hosungs hosungs deleted the storage-connection-string-support branch May 21, 2015 02:11
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

Successfully merging this pull request may close these issues.

3 participants