aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/php/lib
diff options
context:
space:
mode:
authorGravatar Chad Kunde <Kunde21@gmail.com>2017-11-28 20:46:48 -0800
committerGravatar Chad Kunde <Kunde21@gmail.com>2018-01-30 11:09:19 +0800
commit1ac45e116e4e6155b821d25d010f4e70b132827b (patch)
tree820b3cb07d712158124baab08e58899cdeeae274 /src/php/lib
parent0e142d4742bc4449c5fcd4216692fb2f9cb2b37a (diff)
Removed opts checks in BaseStub when the elements aren't not used.
Diffstat (limited to 'src/php/lib')
-rw-r--r--src/php/lib/Grpc/BaseStub.php25
1 files changed, 13 insertions, 12 deletions
diff --git a/src/php/lib/Grpc/BaseStub.php b/src/php/lib/Grpc/BaseStub.php
index 67378a34a8..5f3a96feaa 100644
--- a/src/php/lib/Grpc/BaseStub.php
+++ b/src/php/lib/Grpc/BaseStub.php
@@ -54,6 +54,18 @@ class BaseStub
}
unset($opts['update_metadata']);
}
+ if (!empty($opts['grpc.ssl_target_name_override'])) {
+ $this->hostname_override = $opts['grpc.ssl_target_name_override'];
+ }
+ if ($channel) {
+ if (!is_a($channel, 'Grpc\Channel')) {
+ throw new \Exception('The channel argument is not a'.
+ 'Channel object');
+ }
+ $this->channel = $channel;
+ return;
+ }
+
$package_config = json_decode(
file_get_contents(dirname(__FILE__).'/../../composer.json'), true);
if (!empty($opts['grpc.primary_user_agent'])) {
@@ -61,9 +73,6 @@ class BaseStub
} else {
$opts['grpc.primary_user_agent'] = '';
}
- if (!empty($opts['grpc.ssl_target_name_override'])) {
- $this->hostname_override = $opts['grpc.ssl_target_name_override'];
- }
$opts['grpc.primary_user_agent'] .=
'grpc-php/'.$package_config['version'];
if (!array_key_exists('credentials', $opts)) {
@@ -71,15 +80,7 @@ class BaseStub
'required. Please see one of the '.
'ChannelCredentials::create methods');
}
- if ($channel) {
- if (!is_a($channel, 'Grpc\Channel')) {
- throw new \Exception('The channel argument is not a'.
- 'Channel object');
- }
- $this->channel = $channel;
- } else {
- $this->channel = new Channel($hostname, $opts);
- }
+ $this->channel = new Channel($hostname, $opts);
}
/**