Skip to content

[wip] Guzzle request options - #63

Merged
jamiehannaford merged 3 commits into
php-opencloud:masterfrom
haphan:guzzle-opts
Jun 22, 2016
Merged

jamiehannaford merged 3 commits into
php-opencloud:masterfrom
haphan:guzzle-opts

Conversation

@haphan

@haphan haphan commented Jun 22, 2016

Copy link
Copy Markdown
Collaborator
$requestOptions = [
    'verify' => false,
    'headers' => ['User-Agent' => 'PHP-OPENCLOUD/SDKv1.0']
];

$params = [
    'authUrl' => 'http://172.16.113.139:5000/v3',
    'region'  => 'RegionOne',
    'user'    =>
        array(
            'name' => 'admin',
            'password' => '034cdbdb0fd84c5f',
            'domain'   =>
                array(
                    'id' => 'default',
                ),
        ),
    'requestOptions' => $requestOptions
];


$openstack = new OpenStack\OpenStack($params);

@haphan haphan changed the title Guzzle request options [wip] Guzzle request options Jun 22, 2016
Comment thread src/OpenStack.php Outdated
* ['debugLog'] = (bool) Whether to enable HTTP logging [OPTIONAL]
* ['logger'] = (LoggerInterface) Must set if debugLog is true [OPTIONAL]
* ['messageFormatter'] = (MessageFormatter) Must set if debugLog is true [OPTIONAL]
* ['requestOption'] = (array) Guzzle Http request options [OPTIONAL]

@jamiehannaford jamiehannaford Jun 22, 2016

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/requestOption/requestOptions/

@jamiehannaford

Copy link
Copy Markdown
Contributor

Could we also update the php-opencloud/common dependency to v1.0.5 while we're at it?

@jamiehannaford

Copy link
Copy Markdown
Contributor

php-opencloud/common:v1.0.5 should now be available

@haphan

haphan commented Jun 22, 2016

Copy link
Copy Markdown
Collaborator Author

@jamiehannaford Please review. Thanks.

@jamiehannaford
jamiehannaford merged commit f14f84d into php-opencloud:master Jun 22, 2016
@haphan
haphan deleted the guzzle-opts branch June 22, 2016 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants