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

JGRP-1992 GOOGLE_PING discovery fails if port 443 isn't used #253

Merged
merged 1 commit into from
Dec 10, 2015

Conversation

alanfx
Copy link
Contributor

@alanfx alanfx commented Dec 10, 2015

Change GOOGLE_PING to use port 443 when creating a connection and no
port is specified in the configuration.

Change GOOGLE_PING to use port 443 when creating a connection and no
port is specified in the configuration.
belaban added a commit that referenced this pull request Dec 10, 2015
JGRP-1992 GOOGLE_PING discovery fails if port 443 isn't used
@belaban belaban merged commit 9e2d502 into belaban:master Dec 10, 2015
@belaban
Copy link
Owner

belaban commented Dec 10, 2015

Thanks for the fix, Alan. Did you test this with a real GCE account, or did you use some simulator?

@alanfx
Copy link
Contributor Author

alanfx commented Dec 10, 2015

I tested this with a real GCE account, Bela.

@belaban
Copy link
Owner

belaban commented Dec 10, 2015

OK. I read that there is a Google Cloud server that can be run off of the local file system to check a client's correctness.
Might be worth taking into consideration for future unit testing...

@alanfx
Copy link
Contributor Author

alanfx commented Dec 10, 2015

That would be interesting. Do you have a link with more information?

@belaban
Copy link
Owner

belaban commented Dec 10, 2015

@alanfx
Copy link
Contributor Author

alanfx commented Dec 10, 2015

Thanks a lot, Bela

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.

2 participants