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

FIX SoapClient: PHP 5.6 Soap 1.2 #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pitrrs
Copy link

@pitrrs pitrrs commented Aug 4, 2016

#6 Issue fix (Throwing exception)

@ondrejnov
Copy link
Owner

turn off verify ssl certificate is bad practice and its forbidden in eet documentation at point 6.1:
"The cash register shall, as part of the SSL connection initialisation (SSL handshake) with the tax authority's common technical equipment, mandatorily verify the validity of the SSL server certificate (whether it was issued by a trustworthy authority and whether the name for which it was issued corresponds to the common technical equipment's address."

@pitrrs
Copy link
Author

pitrrs commented Aug 4, 2016

Do you have a proposed solution?

@jakubboucek
Copy link

Exists 3 right ways to fix that:

  1. Add any CA bundle to your project (like a Kdyby/CurlCaBundle) and add bundle to PHP config.
  2. Add certificates package to your server or ask your webhosting to do it.
  3. Get certificate of EET server (or her CA cert) and add it to project configuration.

Thanks @ondrejnov to keep security level :)

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.

3 participants