diff options
author | Chad Kunde <Kunde21@gmail.com> | 2017-11-28 20:46:48 -0800 |
---|---|---|
committer | Chad Kunde <Kunde21@gmail.com> | 2018-01-30 11:09:19 +0800 |
commit | 1ac45e116e4e6155b821d25d010f4e70b132827b (patch) | |
tree | 820b3cb07d712158124baab08e58899cdeeae274 /src/php/lib | |
parent | 0e142d4742bc4449c5fcd4216692fb2f9cb2b37a (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.php | 25 |
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); } /** |