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

Update 01b-docker.md #5

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from
Open

Update 01b-docker.md #5

wants to merge 1 commit into from

Conversation

rob-c
Copy link

@rob-c rob-c commented Aug 30, 2018

Just running through this to setup a docker container to avoid throwing up a full VM to do some work.

I hit a problem, externally to CERN, the URL (command) wrapped in the PR will hang due to the firewall.

This PR is just adding a URL that works externally from CERN so that external users don't have to do proxy-ing.

Not sure if there is a policy on this but new users won't be overly familiar with why things like this don't work from their home institute or on their laptop.

Adding a URL that works externally from CERN
@chrisburr
Copy link
Member

chrisburr commented Aug 30, 2018

@rob-c I always use SSH to access gitlab even when away from CERN so I guess this is due to your institute's firewall rather than the CERN one?

Maybe putting this would be clearer:

This command will fail from some institutes due to their firewall. If this happens you can instead clone using https:

@rob-c
Copy link
Author

rob-c commented Aug 30, 2018

@chrisburr Potentially, I'm traveling on eduroam (and use that in my home institute), and typically find I don't have a problem with the https way of connecting.

Would it be a better idea to re-phrase it as, try this and drop back to https if this fails?

@chrisburr
Copy link
Member

@rob-c Yeah I think that would be good idea. A notable minority of places block non HTTP(S) traffic (i.e. port 80 and 443).

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.

None yet

2 participants