Skip to content
This repository has been archived by the owner on Feb 1, 2021. It is now read-only.

fix typos in docs #2046

Merged
merged 1 commit into from
Apr 6, 2016
Merged

Conversation

allencloud
Copy link
Contributor

Signed-off-by: Sun Hongliang [email protected]

@@ -22,7 +22,7 @@ Swarm clusters in business critical production environments. The following high

## Security

There are many aspects to securing a Docker Swarm cluster. This section covers:
There are many aspects to secure a Docker Swarm cluster. This section covers:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think securing is actually correct here.

Choose a reason for hiding this comment

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

securing is correct

@nishanttotla
Copy link
Contributor

Ping @moxiegirl

@@ -60,16 +60,16 @@ the different components of a Swam cluster listen on. You should use these to
configure your firewalls and other network access control lists.

- **Swarm manager.**
- **Inbound 80/tcp (HTTP)**. This is allows `docker pull` commands to work. If you will be pulling from Docker Hub you will need to allow connections on port 80 from the internet.
- **Inbound 80/tcp (HTTP)**. This allows `docker pull` commands to work. If you will be pulling from Docker Hub you will need to allow connections on port 80 from the internet.

Choose a reason for hiding this comment

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

Change all instances of:

If you will be pulling from Docker Hub you will need to allow connections on port 80 from the internet

to

If you plan to pull images from Docker Hub, you must allow Internet connections through port 80.

@moxiegirl
Copy link

Thanks for the contribution @allencloud --- some small corrections.

@allencloud
Copy link
Contributor Author

Ping @moxiegirl

@thaJeztah
Copy link
Member

LGTM 👍

Signed-off-by: Sun Hongliang <[email protected]>
@moxiegirl
Copy link

Thanks @allencloud !

@moxiegirl moxiegirl merged commit ba9b348 into docker-archive:master Apr 6, 2016
@allencloud allencloud deleted the fix-typos-in-docs branch April 7, 2016 00:40
ChristianKniep pushed a commit to ChristianKniep/swarm that referenced this pull request Jul 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants