unable to add account after installing module

[kabbady]

after install the module, adding account with my username and API key

save button takes along time
after that time return to add account again and all my entries are disappeared

[jonathanspw]

It sounds like you're having connectivity issues with namesilo, or perhaps their api is down at the moment.

Can you please give the return of the following commands?

ping namesilo.org
curl https://www.namesilo.com/api/

[kabbady]

PING namesilo.org (74.206.115.250) 56(84) bytes of data.
64 bytes from 74.206.115.250 (74.206.115.250): icmp_seq=1 ttl=51 time=49.5 ms
64 bytes from 74.206.115.250 (74.206.115.250): icmp_seq=2 ttl=51 time=49.3 ms
64 bytes from 74.206.115.250 (74.206.115.250): icmp_seq=2 ttl=51 time=49.3 ms (DUP!)
64 bytes from 74.206.115.250 (74.206.115.250): icmp_seq=3 ttl=51 time=49.5 ms
64 bytes from 74.206.115.250 (74.206.115.250): icmp_seq=3 ttl=51 time=49.5 ms (DUP!)

===============
curl https://www.namesilo.com/api/
Invalid Request

[julianmatz]

@kabbady Could you also try replacing the code in components/modules/namesilo/apis/namesilo_api.php with the code below?

<?php

require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "namesilo_response.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains_dns.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains_ns.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains_transfer.php";
//require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "account_information.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "users_address.php";

/**
 * Namesilo API processor
 *
 * Documentation on the Namesilo API: http://www.namesilo.com/api_reference.php
 *
 * @copyright Copyright (c) 2013, Phillips Data, Inc.
 * @license http://opensource.org/licenses/mit-license.php MIT License
 * @package namesilo
 */
class NamesiloApi {

	const SANDBOX_URL = "http://sandbox.namesilo.com/api";
	const LIVE_URL = "https://www.namesilo.com/api";

	/**
	 * @var string API version
	 */
	private $api_version = 1;
	/**
	 * @var string The format of the API response
	 */
	private $format = 'xml';
	/**
	 * @var string The user to connect as
	 */
	private $user;
	/**
	 * @var string The username to execute an API command using
	 */
	private $username;
	/**
	 * @var string The key to use when connecting
	 */
	private $key;
	/**
	 * @var boolean Whether or not to process in sandbox mode (for testing)
	 */
	private $sandbox;
	/**
	 * @var array An array representing the last request made
	 */
	private $last_request = array('url' => null, 'args' => null);
    /**
     * @var bool use batch api url if true
     */
	private $batch;
    /**
     * @var int http return code
     */
	public $httpcode;
	
	/**
	 * Sets the connection details
	 *
	 * @param string $user The user to connect as
	 * @param string $key The key to use when connecting
	 * @param boolean $sandbox Whether or not to process in sandbox mode (for testing)
	 * @param string $username The username to execute an API command using
     * @param bool $batch Set true to pass commands to batch API URL.
     *      See https://www.namesilo.com/Support/API-Automated-Batch-Processing
	 */
	public function __construct($user, $key, $sandbox = true, $username = null, $batch=false) {
		
		$this->user = $user;
		$this->key = $key;
		
		$this->sandbox = filter_var( $sandbox, FILTER_VALIDATE_BOOLEAN );
		
		if (!$username)
			$username = $user;
		
		$this->username = $username;

		$this->batch = $batch;
	}
	
	/**
	 * Submits a request to the API
	 *
	 * @param string $command The command to submit
	 * @param array $args An array of key/value pair arguments to submit to the given API command
	 * @return NamesiloResponse The response object
	 */
	public function submit($command, array $args = array()) {

		$url = self::LIVE_URL;
		if ($this->sandbox)
			$url = self::SANDBOX_URL;

		if($this->batch)
            $url = $url . 'batch';
		
		$url .= '/' . $command . "?key={$this->key}";
		
		//$args['ApiUser'] = $this->user;
		//if (!array_key_exists("UserName", $args))
			//$args['UserName'] = $this->username;
		$args['version'] = $this->api_version;
		$args['type'] = $this->format;
		//$args['key'] = $this->key;
		//$args['Command'] = $command;
		
		//if (!isset($args['ClientIP']))
			//$args['ClientIP'] = isset($_SERVER['REMOTE_ADDR']) ? $_SERVER['REMOTE_ADDR'] : "127.0.0.1";
		
		$this->last_request = array(
			'url' => $url,
			'args' => $args
		);
		
		$query = http_build_query( $args );
		$url = $url . "&{$query}";
		
		$ch = curl_init();
		curl_setopt($ch, CURLOPT_URL, $url);
		curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
		#curl_setopt($ch, CURLOPT_POSTFIELDS, $args);
		#curl_setopt($ch, CURLOPT_CUSTOMREQUEST, "POST");
		curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
		curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
		$response = curl_exec($ch);
        $this->httpcode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
		curl_close($ch);
		
		//trigger_error( var_export( $args, true ) );
		//trigger_error( var_export( $response, true ) );
		
		return new NamesiloResponse($response);
	}
	
	/**
	 * Returns the details of the last request made
	 *
	 * @return array An array containg:
	 * 	- url The URL of the last request
	 * 	- args The paramters passed to the URL
	 */
	public function lastRequest() {
		return $this->last_request;
	}
}

[kabbady]

@kabbady Could you also try replacing the code in components/modules/namesilo/apis/namesilo_api.php with the code below?

<?php

require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "namesilo_response.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains_dns.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains_ns.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "domains_transfer.php";
//require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "account_information.php";
require_once dirname( __FILE__ ) . DIRECTORY_SEPARATOR . "commands" . DIRECTORY_SEPARATOR . "users_address.php";

/**
 * Namesilo API processor
 *
 * Documentation on the Namesilo API: http://www.namesilo.com/api_reference.php
 *
 * @copyright Copyright (c) 2013, Phillips Data, Inc.
 * @license http://opensource.org/licenses/mit-license.php MIT License
 * @package namesilo
 */
class NamesiloApi {

	const SANDBOX_URL = "http://sandbox.namesilo.com/api";
	const LIVE_URL = "https://www.namesilo.com/api";

	/**
	 * @var string API version
	 */
	private $api_version = 1;
	/**
	 * @var string The format of the API response
	 */
	private $format = 'xml';
	/**
	 * @var string The user to connect as
	 */
	private $user;
	/**
	 * @var string The username to execute an API command using
	 */
	private $username;
	/**
	 * @var string The key to use when connecting
	 */
	private $key;
	/**
	 * @var boolean Whether or not to process in sandbox mode (for testing)
	 */
	private $sandbox;
	/**
	 * @var array An array representing the last request made
	 */
	private $last_request = array('url' => null, 'args' => null);
    /**
     * @var bool use batch api url if true
     */
	private $batch;
    /**
     * @var int http return code
     */
	public $httpcode;
	
	/**
	 * Sets the connection details
	 *
	 * @param string $user The user to connect as
	 * @param string $key The key to use when connecting
	 * @param boolean $sandbox Whether or not to process in sandbox mode (for testing)
	 * @param string $username The username to execute an API command using
     * @param bool $batch Set true to pass commands to batch API URL.
     *      See https://www.namesilo.com/Support/API-Automated-Batch-Processing
	 */
	public function __construct($user, $key, $sandbox = true, $username = null, $batch=false) {
		
		$this->user = $user;
		$this->key = $key;
		
		$this->sandbox = filter_var( $sandbox, FILTER_VALIDATE_BOOLEAN );
		
		if (!$username)
			$username = $user;
		
		$this->username = $username;

		$this->batch = $batch;
	}
	
	/**
	 * Submits a request to the API
	 *
	 * @param string $command The command to submit
	 * @param array $args An array of key/value pair arguments to submit to the given API command
	 * @return NamesiloResponse The response object
	 */
	public function submit($command, array $args = array()) {

		$url = self::LIVE_URL;
		if ($this->sandbox)
			$url = self::SANDBOX_URL;

		if($this->batch)
            $url = $url . 'batch';
		
		$url .= '/' . $command . "?key={$this->key}";
		
		//$args['ApiUser'] = $this->user;
		//if (!array_key_exists("UserName", $args))
			//$args['UserName'] = $this->username;
		$args['version'] = $this->api_version;
		$args['type'] = $this->format;
		//$args['key'] = $this->key;
		//$args['Command'] = $command;
		
		//if (!isset($args['ClientIP']))
			//$args['ClientIP'] = isset($_SERVER['REMOTE_ADDR']) ? $_SERVER['REMOTE_ADDR'] : "127.0.0.1";
		
		$this->last_request = array(
			'url' => $url,
			'args' => $args
		);
		
		$query = http_build_query( $args );
		$url = $url . "&{$query}";
		
		$ch = curl_init();
		curl_setopt($ch, CURLOPT_URL, $url);
		curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
		#curl_setopt($ch, CURLOPT_POSTFIELDS, $args);
		#curl_setopt($ch, CURLOPT_CUSTOMREQUEST, "POST");
		curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
		curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
		$response = curl_exec($ch);
        $this->httpcode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
		curl_close($ch);
		
		//trigger_error( var_export( $args, true ) );
		//trigger_error( var_export( $response, true ) );
		
		return new NamesiloResponse($response);
	}
	
	/**
	 * Returns the details of the last request made
	 *
	 * @return array An array containg:
	 * 	- url The URL of the last request
	 * 	- args The paramters passed to the URL
	 */
	public function lastRequest() {
		return $this->last_request;
	}
}

Well account added now :)
how can i verify it's working ?

Appericate your help

[julianmatz]

You don't get timeouts anymore? If not, it seems to be working. I noticed an issue with timeouts when using PHP cURL post requests to communicate with Namesilo. I changed the the code to use GET instead of POST, and it seems to work better for some people. I have no idea why, though. I checked the Namesilo documentation to see if any changes were made on their end, but it doesn't seem to be the case.