X Tutup
Skip to content
This repository was archived by the owner on Aug 15, 2022. It is now read-only.

Define user proxy and tunnel URIs in the configuration#11405

Merged
rjeczalik merged 1 commit intomasterfrom
user-proxy
Aug 15, 2017
Merged

Define user proxy and tunnel URIs in the configuration#11405
rjeczalik merged 1 commit intomasterfrom
user-proxy

Conversation

@szkl
Copy link
Member

@szkl szkl commented Aug 14, 2017

Signed-off-by: Sonmez Kartal sonmez@koding.com

Description

Simplify setting of user proxy/tunnel URI.

Motivation and Context

Proxifier module is computing the user proxy/tunnel URIs based on environment runtime.
I've simplified this process by setting the user proxy/tunnel URIs in the configuration of each environment. Proxifier module does not need to have conditional statement branches to figure out what endpoint to use.

How Has This Been Tested?

  • Built an AWS stack on sandbox environment
  • Built a Vagrant stack on sandbox environment

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Signed-off-by: Sonmez Kartal <sonmez@koding.com>
@szkl szkl added the wip label Aug 14, 2017
@szkl szkl requested review from gokmen and rjeczalik August 14, 2017 15:35
@rjeczalik rjeczalik merged commit a189a30 into master Aug 15, 2017
@rjeczalik rjeczalik deleted the user-proxy branch August 15, 2017 10:52
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

X Tutup