Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

Bug 907742 Use 'localhost' exclusively for local port forwarding. #298

Merged
merged 1 commit into from Feb 22, 2013

Conversation

BanzaiMan
Copy link
Contributor

This should fix bug 907742. The behavior change is large enough,
however, that existing documents should be revised.

This should fix bug 907742. The behavior change is large enough,
however, that existing documents should be revised.
@BanzaiMan
Copy link
Contributor Author

[test]

@openshift-bot
Copy link

@openshift-bot
Copy link

Origin Test Results: Running (https://originci-openshift.rhcloud.com/job/test_pull_requests/62/)

@BanzaiMan
Copy link
Contributor Author

I set up an Windows VM to get verify that it works on that platform:

C:\Users\IEUser\DOCUME~1\GitHub\rhc>bundle exec ruby .\bin\rhc port-forward foo
The source :rubygems is deprecated because HTTP requests are insecure.
Please change your source to 'https://rubygems.org' if possible, or 'http://ruby
gems.org' if not.
Password: ************
Checking available ports...
Forwarding ports
  Service Connect to          Forward to
  ====== ============== ==== ================
  httpd  localhost:8080  =>  127.10.28.1:8080
  mysqld localhost:3306  =>  127.10.28.1:3306
Press CTRL-C to terminate port forwarding

The attached image shows the MySQL connection made to localhost's port 3306.

IE9_-Win7 Running

@smarterclayton
Copy link
Contributor

Reviewed, [merge]

@openshift-bot
Copy link

Merge Test Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/merge_pull_requests/1815/) (Image: devenv_2848)

@smarterclayton
Copy link
Contributor

[merge]

@openshift-bot
Copy link

Evaluated up to 5beaa18

openshift-bot pushed a commit that referenced this pull request Feb 22, 2013
@openshift-bot openshift-bot merged commit bc51917 into openshift:master Feb 22, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants