Namesilo and Blesta 4.9 integration

[dipendra]

I installed the namesilo module from github repo of blesta. I set package and setup a sandbox account. It worked .Then i used my real account now every domain search show unavailable. I saw logs n Blesta dashboard but it has shown only three requests though i have done more than that. Also the logs are showing i am requesting for "example.com" domain but i hadnt tried search for this domain.

[Jono]

Yup, so two things.  First, those requests happen when you save the module row details, just to make sure the credentials are valid.  Second, it unfortunately seems the regular availability checks are not being logged.  To add logging to that method you can add the following line at line 2280 of app/components/module/namesilo.php:

        $this->processResponse($api, $result);

 

[dipendra]

Thanks i will add logging and see what is happening. ❤️

[dipendra]

1 hour ago, Jono said:

Yup, so two things.  First, those requests happen when you save the module row details, just to make sure the credentials are valid.  Second, it unfortunately seems the regular availability checks are not being logged.  To add logging to that method you can add the following line at line 2280 of app/components/module/namesilo.php:

        $this->processResponse($api, $result);

 

There are comments in this line can u tell me inside which function should i insert this line?

[dipendra]

Okay, Namesilo is returning the value correctly i inserted this line to checkAvailability function and API request is fine and the data returned is also correct. So the problem is in between the Module fetched data to result display page.

[Deactivat3d]

Hi friend,

I'd be interested to know what's causing this and whether it's the same problem I'm experiencing. Every domain turns "Sorry, that domain is not available." The transfer option seems to work, however. Slight differences in our log display also have me intrigued. I was previously using the pre-forked version and since upgraded to 1.10.1.

 

[nahanil]

@SmallWeb Which version of PHP are you using when the issue occurs? This is a known issue with the XML parser in PHP 7.3.17 and 7.4.5 specifically and not neccessarily Blesta itself https://bugs.php.net/bug.php?id=79528

There is currently an open pull request for the namesilo module that fixes it https://github.com/blesta/module-namesilo/pull/9 and the next PHP releases should also fix the issue with the XML parser, but we will likely have to wait until early next week for the changes to be merged into the blesta module's master branch (cant work Jono too hard, he's probably already got too much on his plate) 

https://github.com/php/php-src/commit/cb265a0addf15c5cf0c029f167a9de030150cf15
https://github.com/php/php-src/commit/54148fd68677e8b13b5a11c309c601d1248fb78b

[Deactivat3d]

1 minute ago, nahanil said:

@SmallWeb Which version of PHP are you using when the issue occurs? This is a known issue with PHP 7.3.17 and 7.4.5 specifically and not neccessarily Blesta itself https://bugs.php.net/bug.php?id=79528

There is currently an open pull request for the namesilo module that fixes it https://github.com/blesta/module-namesilo/pull/9 and the next PHP releases should also fix the issue with the XML parser, but we will likely have to wait until early next week for the changes to be merged into the blesta module's master branch (cant work Jono too hard, he's probably already got too much on his plate) 

https://github.com/php/php-src/commit/cb265a0addf15c5cf0c029f167a9de030150cf15
https://github.com/php/php-src/commit/54148fd68677e8b13b5a11c309c601d1248fb78b

Thank you, it is PHP 7.3.17.

Based on the latest Blesta release notes and core developments, I would say every one is working very hard. Happy to be patient on this one