-
Notifications
You must be signed in to change notification settings - Fork 732
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Not able to enable https for AwsAuthV4 transport #1456
Comments
It could be an option to pass the ssl key to the config array or maybe we could also use getParam to fetch the ssl setting, as it is done with the access keys. |
@cfoehrdes Feel free to open a PR for further discussions. This should make it more obvious what the changes required would be. |
Closing as #1457 was merged. |
@ruflin Seems to have been never closed. ;) |
Ups, seems like I hit the wrong button. Thanks @aderuwe |
We are trying to use the
AwsAuthV4
transport with ssl and are not able to pass the ssl option to it.It seems that it is not possible because the
ssl
config param accessed in theAwsAuthV4
class (see https://github.com/ruflin/Elastica/blob/master/lib/Elastica/Transport/AwsAuthV4.php#L77) is never set.The keys of the config array are limited to
bigintConversion
,curl
,headers
andurl
in the client when preparing the transport configuration. (see https://github.com/ruflin/Elastica/blob/master/lib/Elastica/Client.php#L162)The text was updated successfully, but these errors were encountered: