Examples: Pachubeclient + chatServer = not working

Hi!

I'm new to arduino.

I want to make an automatic plant watering system. two important functions are:
-send data to cosm
-telnet client to change setpoint for soil moist

So i have tried to combine the examples for cosmClient og chatServer.

 /*
  Cosm sensor client
 
 This sketch connects an analog sensor to Cosm (http://www.cosm.com)
 using a Wiznet Ethernet shield. You can use the Arduino Ethernet shield, or
 the Adafruit Ethernet shield, either one will work, as long as it's got
 a Wiznet Ethernet module on board.
 
 This example has been updated to use version 2.0 of the Cosm.com API. 
 To make it work, create a feed with a datastream, and give it the ID
 sensor1. Or change the code below to match your feed.
 
 
 Circuit:
 * Analog sensor attached to analog in 0
 * Ethernet shield attached to pins 10, 11, 12, 13
 
 created 15 March 2010
 updated 16 Mar 2012
 by Tom Igoe with input from Usman Haque and Joe Saavedra
 
http://arduino.cc/en/Tutorial/PachubeClient
 This code is in the public domain.
 
 */

#include <SPI.h>
#include <Ethernet.h>

#define APIKEY         "of3ryVkzitUd7Sz9wdXXJ65Z0-2SAKw2djZKNFl2UW1VWT0g" // your cosm api key
#define FEEDID         61758 // your feed ID
#define USERAGENT      "Cosm Arduino Example (61758)" // user agent is the project name

// assign a MAC address for the ethernet controller.
// Newer Ethernet shields have a MAC address printed on a sticker on the shield
// fill in your address here:
byte mac[] = { 0x90, 0xa2, 0xda, 0x00, 0x78, 0x08 };

// fill in an available IP address on your network here,
// for manual configuration:
IPAddress ip(10,0,1,20);
// initialize the library instance:
EthernetClient cosmclient;

// if you don't want to use DNS (and reduce your sketch size)
// use the numeric IP instead of the name for the server:
IPAddress cosmserver(216,52,233,121);      // numeric IP for api.cosm.com
//char server[] = "api.cosm.com";   // name address for cosm API

unsigned long lastConnectionTime = 0;          // last time you connected to the server, in milliseconds
boolean lastConnected = false;                 // state of the connection last time through the main loop
const unsigned long postingInterval = 10*1000; //delay between updates to Cosm.com

// telnet defaults to port 23
EthernetServer telnetserver(23);
boolean gotAMessage = false; // whether or not you got a message from the client yet



void setup() {
  // start serial port:
  Serial.begin(9600);
 // start the Ethernet connection:
  if (Ethernet.begin(mac) == 0) {
    Serial.println("Failed to configure Ethernet using DHCP");
    // DHCP failed, so use a fixed IP address:
    Ethernet.begin(mac, ip);
  }
 
 //Telnet config 
  IPAddress ip(192,168,0, 150);
  IPAddress gateway(192,168,0, 1);
  IPAddress subnet(255, 255, 255, 0);

  Ethernet.begin(mac, ip, gateway, subnet);
  // start listening for clients
  telnetserver.begin();
}

void loop() {
  
   EthernetClient telnetclient = telnetserver.available();
  
  // when the client sends the first byte, say hello:
  if (telnetclient) {
    if (!gotAMessage) {
      Serial.println("We have a new client");
      telnetclient.println("Hello, client!"); 
      gotAMessage = true;
    }
    
    // read the bytes incoming from the client:
    char thisChar = telnetclient.read();
    // echo the bytes back to the client:
    telnetserver.write(thisChar);
    // echo the bytes to the server as well:
    Serial.print(thisChar);
  }
  
  // read the analog sensor:
  int sensorReading = analogRead(A0);   

  // if there's incoming data from the net connection.
  // send it out the serial port.  This is for debugging
  // purposes only:
  if (cosmclient.available()) {
    char c = cosmclient.read();
    Serial.print(c);
  }

  // if there's no net connection, but there was one last time
  // through the loop, then stop the client:
  if (!cosmclient.connected() && lastConnected) {
    Serial.println();
    Serial.println("disconnecting.");
    cosmclient.stop();
  }

  // if you're not connected, and ten seconds have passed since
  // your last connection, then connect again and send data:
  if(!cosmclient.connected() && (millis() - lastConnectionTime > postingInterval)) {
    sendData(sensorReading);
  }
  // store the state of the connection for next time through
  // the loop:
  lastConnected = cosmclient.connected();
}

// this method makes a HTTP connection to the server:
void sendData(int thisData) {
  // if there's a successful connection:
  if (cosmclient.connect(cosmserver, 80)) {
    Serial.println("connecting...");
    // send the HTTP PUT request:
    cosmclient.print("PUT /v2/feeds/");
    cosmclient.print(FEEDID);
    cosmclient.println(".csv HTTP/1.1");
    cosmclient.println("Host: api.cosm.com");
    cosmclient.print("X-ApiKey: ");
    cosmclient.println(APIKEY);
    cosmclient.print("User-Agent: ");
    cosmclient.println(USERAGENT);
    cosmclient.print("Content-Length: ");

    // calculate the length of the sensor reading in bytes:
    // 8 bytes for "sensor1," + number of digits of the data:
    int thisLength = 8 + getLength(thisData);
    cosmclient.println(thisLength);

    // last pieces of the HTTP PUT request:
    cosmclient.println("Content-Type: text/csv");
    cosmclient.println("Connection: close");
    cosmclient.println();

    // here's the actual content of the PUT request:
    cosmclient.print("sensor1,");
    cosmclient.println(thisData);
  
  } 
  else {
    // if you couldn't make a connection:
    Serial.println("connection failed");
    Serial.println();
    Serial.println("disconnecting.");
    cosmclient.stop();
  }
   // note the time that the connection was made or attempted:
  lastConnectionTime = millis();
}


// This method calculates the number of digits in the
// sensor reading.  Since each digit of the ASCII decimal
// representation is a byte, the number of digits equals
// the number of bytes:

int getLength(int someValue) {
  // there's at least one byte:
  int digits = 1;
  // continually divide the value by ten, 
  // adding one to the digit count for each
  // time you divide, until you're at 0:
  int dividend = someValue /10;
  while (dividend > 0) {
    dividend = dividend /10;
    digits++;
  }
  // return the number of digits:
  return digits;
}

Over the serialport I get this:

connection failed

disconnecting.
We have a new client
ÿûÿû ÿûÿû'ÿýÿûÿýÿþÿþ ÿþÿþ'ÿüÿû$ÿþ$tset
connection failed

disconnecting.
connection failed

disconnecting.
connection failed

disconnecting.
connection failed

disconnecting.
connection failed

disconnecting.
connection failed

disconnecting.
connection failed

The tset was a typo, trying to write test, using putty

No API requests on cosm.com

How can I make the cosm part work? It looks like the telnet server is working.. (exept for the strange letters: ÿûÿû ÿûÿû'ÿýÿûÿýÿþÿþ ÿþÿþ'ÿüÿû$ÿþ$)

Please help a newbie with some code!

regards Magnus
Norway

Nowbody knows/understands whats wrong??

I know I can use 4 sockets, but can I run 2 clients and 1 server at the same time? Could this be the problem?

I'm using

Ethernet.begin(mac, ip, gateway, subnet);

but the documentation says

Ethernet.begin(mac, ip, dns, gateway, subnet);

Could this be the problem? or does the method understand thar i skiped dns?

Magnus

I have even tryed to update the code, using stop av flush:

/* Circuit:
 * Analog sensor attached to analog in 0
 * Ethernet shield attached to pins 10, 11, 12, 13
 
 * Kjører 2 klienter
   -telnetclient
   -cosmclient
 * Kjører 1 server
   - cosmserver
 
 */

#include <SPI.h>
#include <Ethernet.h>

#define APIKEY         "API" // your cosm api key
#define FEEDID         00000// your feed ID
#define USERAGENT      "Cosm Arduino Example (00000)" // user agent is the project name

byte mac[] = { 0x90, 0xa2, 0xda, 0x00, 0x78, 0x08 };
IPAddress ip(192,168,0, 150);
IPAddress gateway(192,168,0, 1);
IPAddress subnet(255, 255, 255, 0);

// initialize the library instance:
EthernetClient cosmclient;

IPAddress server(216,52,233,121);      // numeric IP for api.cosm.com

unsigned long lastConnectionTime = 0;          // last time you connected to the server, in milliseconds
boolean lastConnected = false;                 // state of the connection last time through the main loop
const unsigned long postingInterval = 10*1000; //delay between updates to Cosm.com

// telnet defaults to port 23
EthernetServer telnetserver(1814);
boolean gotAMessage = false; // whether or not you got a message from the client yet



void setup() {
  // start serial port:
  Serial.begin(9600);
  // start the Ethernet connection:
  Ethernet.begin(mac, ip, gateway, subnet);
  // start listening for clients
  telnetserver.begin();
}

void loop() {
  
  EthernetClient telnetclient = telnetserver.available();
  
  // when the client sends the first byte, say hello:
  if (telnetclient) {
    cosmclient.flush();
    cosmclient.stop();
    if (gotAMessage  == false) {
      Serial.println("We have a new client");
      telnetclient.println("Hello, client!"); 
      gotAMessage = true;
    }
    // read the bytes incoming from the client:
    char thisChar = telnetclient.read();
    // echo the bytes back to the client:
    telnetserver.write(thisChar);
    // echo the bytes to the server as well:
    Serial.print(thisChar);
    if (thisChar == 0x0d){
      Serial.println("Bye, client");
      telnetclient.flush();
      telnetclient.stop();
    }
  }
   
  
  // if there's no net connection, but there was one last time
  // through the loop, then stop the client:
  if (cosmclient.connected() == false && lastConnected) {
    Serial.println();
    Serial.println("disconnecting2.");
    cosmclient.flush();
    cosmclient.stop();
  }

  // if you're not connected, and ten seconds have passed since
  // your last connection, then connect again and send data:
  if((millis() - lastConnectionTime > postingInterval)) { // && telnetclient.connected() == f
    telnetclient.flush();
    telnetclient.stop();
    cosmclient.flush();
    cosmclient.stop();
    Ethernet.begin(mac, ip);  
    if(cosmclient.connected() == false){
      // read the analog sensor:
      int sensorReading = analogRead(A0);   
      sendData(sensorReading);
    }
  }
  // store the state of the connection for next time through
  // the loop:
  lastConnected = cosmclient.connected();
}

// this method makes a HTTP connection to the server:
void sendData(int thisData) {
    
  // if there's a successful connection:
  if (cosmclient.connect(server, 80)) { // kobler til, og klarer det
    Serial.println("connecting...");
    // send the HTTP PUT request:
    cosmclient.print("PUT /v2/feeds/");
    cosmclient.print(FEEDID);
    cosmclient.println(".csv HTTP/1.1");
    cosmclient.println("Host: api.cosm.com");
    cosmclient.print("X-ApiKey: ");
    cosmclient.println(APIKEY);
    cosmclient.print("User-Agent: ");
    cosmclient.println(USERAGENT);
    cosmclient.print("Content-Length: ");

    // calculate the length of the sensor reading in bytes:
    // 8 bytes for "sensor1," + number of digits of the data:
    int thisLength = 8 + getLength(thisData);
    cosmclient.println(thisLength);

    // last pieces of the HTTP PUT request:
    cosmclient.println("Content-Type: text/csv");
    cosmclient.println("Connection: close");
    cosmclient.println();

    // here's the actual content of the PUT request:
    cosmclient.print("sensor1,");
    cosmclient.println(thisData);
    cosmclient.flush();
    cosmclient.stop();
  
  } 
  else {
    // if you couldn't make a connection:
    Serial.println("connection failed");
    Serial.println();
    Serial.println("disconnecting1.");
    cosmclient.stop();
  }
   // note the time that the connection was made or attempted:
  lastConnectionTime = millis();
}


// This method calculates the number of digits in the
// sensor reading.  Since each digit of the ASCII decimal
// representation is a byte, the number of digits equals
// the number of bytes:

int getLength(int someValue) {
  // there's at least one byte:
  int digits = 1;
  // continually divide the value by ten, 
  // adding one to the digit count for each
  // time you divide, until you're at 0:
  int dividend = someValue /10;
  while (dividend > 0) {
    dividend = dividend /10;
    digits++;
  }
  // return the number of digits:
  return digits;
}