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

go-tour: sandbox slide needs updating #7253

Closed
gopherbot opened this issue Feb 3, 2014 · 6 comments
Closed

go-tour: sandbox slide needs updating #7253

gopherbot opened this issue Feb 3, 2014 · 6 comments
Milestone

Comments

@gopherbot
Copy link
Contributor

by ghchinoy:

What steps will reproduce the problem?
If possible, include a link to a program on play.golang.org.
1. http://tour.golang.org/#3
2. run the example
3. result has a warning/error: "<nil> dial tcp: missing port in address
google.com"

Suggested fix:

Change tour/#3's sample program 
from:
fmt.Println(net.Dial("tcp", "google.com"))

to:
fmt.Println(net.Dial("tcp", "google.com:80"))

or:
fmt.Println(net.Dial("tcp", "google.com:443"))

What is the expected output?

Something like: &{{0xc084039000}} <nil>

What do you see instead?

<nil> dial tcp: missing port in address google.com

Which compiler are you using (5g, 6g, 8g, gccgo)?
N/A - tour

Which operating system are you using?
N/A - tour

Which version are you using?  (run 'go version')
tour's 1.2rc3

Please provide any additional information below.
@dominikh
Copy link
Member

dominikh commented Feb 3, 2014

Comment 1:

More specifically, that whole slide is out of date because file access and network
access are possible to some degree now, thanks to the NaCl port the playground uses.

@gopherbot
Copy link
Contributor Author

@adg
Copy link
Contributor

adg commented Feb 4, 2014

Comment 3:

Owner changed to @adg.

Status changed to Accepted.

@adg
Copy link
Contributor

adg commented Feb 4, 2014

Comment 4:

https://golang.org/cl/59990044

Status changed to Started.

@rsc
Copy link
Contributor

rsc commented Mar 3, 2014

Comment 5:

Labels changed: added priority-go1, repo-tour.

@adg
Copy link
Contributor

adg commented Mar 3, 2014

Comment 6:

Status changed to Fixed.

@rsc rsc added this to the Go1 milestone Apr 10, 2015
@rsc rsc removed the priority-go1 label Apr 10, 2015
@golang golang locked and limited conversation to collaborators Jun 25, 2016
@rsc rsc unassigned adg Jun 23, 2022
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants