tomolimo / processmaker

GLPI plugin that provides an interface with ProcessMaker (http://www.processmaker.com/)
30 stars 11 forks source link

Add configuration option for SoapClient #52

Closed orouet closed 5 years ago

orouet commented 6 years ago

We are using a local CA so we must set a local certificate for SoapClient or we must entirely disable SSL checks. We do so on editing openSoap function in processmaker.inc.php Could you add an option in the configuration form to disable SSL checks ?

tomolimo commented 6 years ago

Hello @orouet I'm not sure to understand you when you speak about CA? What would you like to do with SSL? Thank you, Regards, Tomolimo

orouet commented 6 years ago

Yes SSL/TLS You could also add an option to set the public Base URL of PM which can be different from the private Base URL used by the SoapClient.

tomolimo commented 6 years ago

You could also add an option to set the public Base URL of PM which can be different from the private Base URL used by the SoapClient.

No it cannot be different, otherwise the server will not be able to work with HTTP_REFERER information.

tomolimo commented 6 years ago

We are using a local CA so we must set a local certificate for SoapClient or we must entirely disable SSL checks.

Do yo mean that you have set SSL on the PM server?

orouet commented 6 years ago

Yes SSL on the PM server and on the GLPI server. If we don't use SSL on PM server, we have Mixed Content Error

tomolimo commented 6 years ago

Hello @orouet I didn't forget you, but I'm working in a company and I've also other topics to manage.

So to come back to your request, and especially:

You could also add an option to set the public Base URL of PM which can be different from the private Base URL used by the SoapClient.

Could you give me more details about this sentence?

Thank you, Regards, Tomolimo

tomolimo commented 5 years ago

We do so on editing openSoap function in processmaker.inc.php

Hello @orouet, I'm in the process of adding this feature, to help me doing that, could you post your code? Thank you Regards, Tomolimo

tomolimo commented 5 years ago

Fixed in https://github.com/tomolimo/processmaker/releases/tag/3.3.8 and in https://github.com/tomolimo/processmaker/releases/tag/3.4.5