Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Jan 8, 2016

Also changing how the variable is used. Now, pre-pending the protocol in front. (Assumes the user does something like localhost:8888.)

/cc @hbchai

@dhermes dhermes added the api: pubsub Issues related to the Pub/Sub API. label Jan 8, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 8, 2016
Also changing how the variable is used. Now, pre-pending the
protocol in front. (Assumes the user does something like
`localhost:8888`.)
@dhermes dhermes force-pushed the pubsub-emulator-host branch from 9d45ce0 to 03dfdf7 Compare January 8, 2016 21:53

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@tseaver
Copy link
Contributor

tseaver commented Jan 14, 2016

LGTM

dhermes added a commit that referenced this pull request Jan 14, 2016
Changing use of PUBSUB_HOST env. var. to PUBSUB_EMULATOR_HOST.
@dhermes dhermes merged commit 3a2a1c5 into googleapis:master Jan 14, 2016
@dhermes
Copy link
Contributor Author

dhermes commented Jan 14, 2016

@hbchai I also updated gcloud and confirmed that the new settings were present. Still need to test that it works though

@dhermes dhermes deleted the pubsub-emulator-host branch January 14, 2016 22:06
parthea pushed a commit that referenced this pull request Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants