Skip to content
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

Added a way to assign CA certificate path #10

Merged
merged 4 commits into from
Mar 20, 2019
Merged

Added a way to assign CA certificate path #10

merged 4 commits into from
Mar 20, 2019

Conversation

sudiptpa
Copy link
Contributor

No description provided.

@sudiptpa
Copy link
Contributor Author

@rotassator I am using this package with Laravel v4.2, we are now getting error saying :

"[curl] 60: SSL certificate problem: unable to get local issuer certificate [url] https://api.payway.com.au/rest/v1/single-use-tokens"

Now, as a hot fix i have done changes in my server.

Downloaded CA cert from : https://curl.haxx.se/docs/caextract.html

and assigned it as:

        $gateway = Omnipay::create('PaywayRest_DirectDebit');

        $gateway->setApiKeyPublic($settings->apiKeyPublic);
        $gateway->setApiKeySecret($settings->apiKeySecret);
        $gateway->setMerchantId($settings->merchantId);
        $gateway->setTestMode($credential->is_test);

        $gateway->setSSLCertificatePath(app_path('certificate/cacert.pem'));

Could you please review and merge it.

@sudiptpa
Copy link
Contributor Author

@rotassator I have already tested on our production and it started working, it would be great if you could tag a new release.

@sudiptpa
Copy link
Contributor Author

sudiptpa commented Mar 2, 2019

Ping @rotassator I was waiting for this PR to be reviewed and merged.

Copy link
Owner

@rotassator rotassator left a comment

Choose a reason for hiding this comment

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

Looks good!

@rotassator rotassator merged commit 4241c1f into rotassator:master Mar 20, 2019
@sudiptpa
Copy link
Contributor Author

@rotassator the 1.9.0 is not synced to packagist.org yet, do you have web hook setup for automatic syncing or releases and code updates?

@rotassator
Copy link
Owner

Ah. I'd initially set up the Packagist integration via Github services. They've been deprecated now in favour of webhooks, which I've set up now. Everything should be up to date.

Thanks for letting me know. :)

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