Skip to content

Conversation

iteman
Copy link
Member

@iteman iteman commented Jul 13, 2016

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License BSD-2-Clause

This PR provides a feature that allows you to specify the proxy URL with the port number as the following:

...
phpmentors_proxy_url_rewrite:
        proxy_urls:
            foo_bar:
                path: "!^.*!"
                proxy_url: "http://www.example.com:8080/foo/bar"

iteman added 2 commits July 13, 2016 13:28
…number as the following:

phpmentors_proxy_url_rewrite:
    proxy_urls:
        foo_bar:
            path: "!^.*!"
            proxy_url: "http://www.example.com:8080/foo/bar"
@iteman iteman self-assigned this Jul 13, 2016
@iteman iteman changed the title Proxy URL with the port number A proxy URL with the port number Jul 13, 2016
@iteman iteman merged commit bce0bd2 into phpmentors-jp:master Jul 13, 2016
iteman added a commit that referenced this pull request Jul 13, 2016
This PR was merged into the 1.2-dev branch.

Discussion
----------

A proxy URL with the port number

| Q             | A
| ------------- | ---
| Branch?       | `master`
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | BSD-2-Clause

This PR provides a feature that allows you to specify the proxy URL with the port number as the following:

```yaml
...
phpmentors_proxy_url_rewrite:
        proxy_urls:
            foo_bar:
                path: "!^.*!"
                proxy_url: "http://www.example.com:8080/foo/bar"
```

Commits
-------

54dd0b4 add a feature that allows you to specify the proxy URL with the port number as the following:
bce0bd2 fix CS
@iteman iteman deleted the proxy-url-with-port-number branch July 13, 2016 05:09
@iteman iteman added this to the 1.2.0 milestone Jul 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant