Code

Added sanity check to beware of broken ldap config on client.
[gosa.git] / gosa-si / modules / GosaPackages.pm
index eea707322b1140f74b22f5d9ca1cd3b0c4d928cc..d23b658e3950f41bc00b57263b51e6b8d9560729 100644 (file)
@@ -13,15 +13,19 @@ use Data::Dumper;
 use GOSA::DBsqlite;
 use MIME::Base64;
 
+my $event_dir = "/usr/lib/gosa-si/server/events";
+use lib "/usr/lib/gosa-si/server/events";
+
 BEGIN{}
 END{}
 
-my ($server_activ, $server_port, $server_passwd, $max_clients, $server_event_dir);
+my ($server_activ, $server_ip, $server_mac_address, $server_port, $server_passwd, $max_clients, $server_event_dir);
 my ($bus_activ, $bus_passwd, $bus_ip, $bus_port);
-my ($gosa_activ, $gosa_ip, $gosa_port, $gosa_passwd);
+my ($gosa_activ, $gosa_ip, $gosa_mac_address, $gosa_port, $gosa_passwd, $network_interface);
 my ($job_queue_timeout, $job_queue_file_name);
 
 my $gosa_server;
+my $event_hash;
 
 my %cfg_defaults = 
 ("general" =>
@@ -29,20 +33,20 @@ my %cfg_defaults =
     },
 "server" =>
     {"server_activ" => [\$server_activ, "on"],
+    "server_ip" => [\$server_ip, "0.0.0.0"],
     "server_port" => [\$server_port, "20081"],
     "server_passwd" => [\$server_passwd, ""],
     "max_clients" => [\$max_clients, 100],
-    "server_event_dir" => [\$server_event_dir, '/usr/lib/gosa-si/server/events'],
     },
 "bus" =>
     {"bus_activ" => [\$bus_activ, "on"],
     "bus_passwd" => [\$bus_passwd, ""],
-    "bus_ip" => [\$bus_ip, ""],
+    "bus_ip" => [\$bus_ip, "0.0.0.0"],
     "bus_port" => [\$bus_port, "20080"],
     },
 "gosa" =>
     {"gosa_activ" => [\$gosa_activ, "on"],
-    "gosa_ip" => [\$gosa_ip, ""],
+    "gosa_ip" => [\$gosa_ip, "0.0.0.0"],
     "gosa_port" => [\$gosa_port, "20082"],
     "gosa_passwd" => [\$gosa_passwd, "none"],
     },
@@ -53,9 +57,8 @@ my %cfg_defaults =
 
 # read configfile and import variables
 &read_configfile();
-
-# detect own ip and mac address
-my ($server_ip, $server_mac_address) = &get_ip_and_mac(); 
+$network_interface= &get_interface_for_ip($server_ip);
+$gosa_mac_address= &get_mac($network_interface);
 
 # complete addresses
 my $server_address = "$server_ip:$server_port";
@@ -63,31 +66,12 @@ my $bus_address = "$bus_ip:$bus_port";
 my $gosa_address = "$gosa_ip:$gosa_port";
 
 # create general settings for this module
-my $gosa_cipher = &create_ciphering($gosa_passwd);
+#y $gosa_cipher = &create_ciphering($gosa_passwd);
 my $xml = new XML::Simple();
 
-# open gosa socket
-if ($gosa_activ eq "on") {
-    &main::daemon_log(" ",1);
-    $gosa_server = IO::Socket::INET->new(LocalPort => $gosa_port,
-            Type => SOCK_STREAM,
-            Reuse => 1,
-            Listen => 1,
-            );
-    if (not defined $gosa_server) {
-        &main::daemon_log("cannot start tcp server at $gosa_port for communication to gosa: $@", 1);
-    } else {
-        &main::daemon_log("start server for communication to gosa: $gosa_address", 1);
-        
-    }
-}
-
-# create gosa job queue as a SQLite DB 
-my $table_name = "jobs";
-my $sqlite = GOSA::DBsqlite->new($job_queue_file_name);
-
-
 
+# import events
+&import_events();
 
 ## FUNCTIONS #################################################################
 
@@ -127,56 +111,152 @@ sub read_configfile {
     }
 }
 
+#===  FUNCTION  ================================================================
+#         NAME:  get_interface_for_ip
+#   PARAMETERS:  ip address (i.e. 192.168.0.1)
+#      RETURNS:  array: list of interfaces if ip=0.0.0.0, matching interface if found, undef else
+#  DESCRIPTION:  Uses proc fs (/proc/net/dev) to get list of interfaces.
+#===============================================================================
+sub get_interface_for_ip {
+        my $result;
+        my $ip= shift;
+        if ($ip && length($ip) > 0) {
+                my @ifs= &get_interfaces();
+                if($ip eq "0.0.0.0") {
+                        $result = "all";
+                } else {
+                        foreach (@ifs) {
+                                my $if=$_;
+                                if(get_ip($if) eq $ip) {
+                                        $result = $if;
+                                }
+                        }       
+                }
+        }       
+        return $result;
+}
 
 #===  FUNCTION  ================================================================
-#         NAME:  get_ip_and_mac 
-#   PARAMETERS:  nothing
-#      RETURNS:  (ip, mac) 
-#  DESCRIPTION:  executes /sbin/ifconfig and parses the output, the first occurence 
-#                of a inet address is returned as well as the mac address in the line
-#                above the inet address
+#         NAME:  get_interfaces 
+#   PARAMETERS:  none
+#      RETURNS:  (list of interfaces) 
+#  DESCRIPTION:  Uses proc fs (/proc/net/dev) to get list of interfaces.
 #===============================================================================
-sub get_ip_and_mac {
-    my $ip = "0.0.0.0.0"; # Defualt-IP
-    my $mac = "00:00:00:00:00:00";  # Default-MAC
-    my @ifconfig = qx(/sbin/ifconfig);
-    foreach(@ifconfig) {
-        if (/Hardware Adresse (\S{2}):(\S{2}):(\S{2}):(\S{2}):(\S{2}):(\S{2})/) {
-            $mac = "$1:$2:$3:$4:$5:$6";
-            next;
-        }
-        if (/inet Adresse:(\d+).(\d+).(\d+).(\d+)/) {
-            $ip = "$1.$2.$3.$4";
-            last;
+sub get_interfaces {
+        my @result;
+        my $PROC_NET_DEV= ('/proc/net/dev');
+
+        open(PROC_NET_DEV, "<$PROC_NET_DEV")
+                or die "Could not open $PROC_NET_DEV";
+
+        my @ifs = <PROC_NET_DEV>;
+
+        close(PROC_NET_DEV);
+
+        # Eat first two line
+        shift @ifs;
+        shift @ifs;
+
+        chomp @ifs;
+        foreach my $line(@ifs) {
+                my $if= (split /:/, $line)[0];
+                $if =~ s/^\s+//;
+                push @result, $if;
         }
-    }
-    return ($ip, $mac);
+
+        return @result;
 }
 
+#===  FUNCTION  ================================================================
+#         NAME:  get_mac 
+#   PARAMETERS:  interface name (i.e. eth0)
+#      RETURNS:  (mac address) 
+#  DESCRIPTION:  Uses ioctl to get mac address directly from system.
+#===============================================================================
+sub get_mac {
+        my $ifreq= shift;
+        my $result;
+        if ($ifreq && length($ifreq) > 0) { 
+                if($ifreq eq "all") {
+                        $result = "00:00:00:00:00:00";
+                } else {
+                        my $SIOCGIFHWADDR= 0x8927;     # man 2 ioctl_list
+
+                        # A configured MAC Address should always override a guessed value
+                        if ($gosa_mac_address and length($gosa_mac_address) > 0) {
+                                $result= $gosa_mac_address;
+                        }
+
+                        socket SOCKET, PF_INET, SOCK_DGRAM, getprotobyname('ip')
+                                or die "socket: $!";
+
+                        if(ioctl SOCKET, $SIOCGIFHWADDR, $ifreq) {
+                                my ($if, $mac)= unpack 'h36 H12', $ifreq;
+
+                                if (length($mac) > 0) {
+                                        $mac=~ m/^([0-9a-f][0-9a-f])([0-9a-f][0-9a-f])([0-9a-f][0-9a-f])([0-9a-f][0-9a-f])([0-9a-f][0-9a-f])([0-9a-f][0-9a-f])$/;
+                                        $mac= sprintf("%s:%s:%s:%s:%s:%s", $1, $2, $3, $4, $5, $6);
+                                        $result = $mac;
+                                }
+                        }
+                }
+        }
+        return $result;
+}
 
 #===  FUNCTION  ================================================================
-#         NAME:  open_socket
-#   PARAMETERS:  PeerAddr string something like 192.168.1.1 or 192.168.1.1:10000
-#                [PeerPort] string necessary if port not appended by PeerAddr
-#      RETURNS:  socket IO::Socket::INET
-#  DESCRIPTION:  open a socket to PeerAddr
+#         NAME:  get_ip 
+#   PARAMETERS:  interface name (i.e. eth0)
+#      RETURNS:  (ip address) 
+#  DESCRIPTION:  Uses ioctl to get ip address directly from system.
 #===============================================================================
-sub open_socket {
-    my ($PeerAddr, $PeerPort) = @_ ;
-    if(defined($PeerPort)){
-        $PeerAddr = $PeerAddr.":".$PeerPort;
+sub get_ip {
+        my $ifreq= shift;
+        my $result= "";
+        my $SIOCGIFADDR= 0x8915;       # man 2 ioctl_list
+        my $proto= getprotobyname('ip');
+
+        socket SOCKET, PF_INET, SOCK_DGRAM, $proto
+                or die "socket: $!";
+
+        if(ioctl SOCKET, $SIOCGIFADDR, $ifreq) {
+                my ($if, $sin)    = unpack 'a16 a16', $ifreq;
+                my ($port, $addr) = sockaddr_in $sin;
+                my $ip            = inet_ntoa $addr;
+
+                if ($ip && length($ip) > 0) {
+                        $result = $ip;
+                }
+        }
+
+        return $result;
+}
+
+
+sub import_events {
+    if (not -e $event_dir) {
+        daemon_log("ERROR: cannot find directory or directory is not readable: $event_dir", 1);   
     }
-    my $socket;
-    $socket = new IO::Socket::INET(PeerAddr => $PeerAddr ,
-            Porto => "tcp" ,
-            Type => SOCK_STREAM,
-            Timeout => 5,
-            );
-    if(not defined $socket) {
-        return;
+    opendir (DIR, $event_dir) or die "ERROR while loading gosa-si-events from directory $event_dir : $!\n";
+
+    while (defined (my $event = readdir (DIR))) {
+        if( $event eq "." || $event eq ".." ) { next; }    
+
+        eval{ require $event; };
+        if( $@ ) {
+            &main::daemon_log("import of event module '$event' failed", 1);
+            &main::daemon_log("$@", 8);
+            next;
+        }
+
+        $event =~ /(\S*?).pm$/;
+        my $event_module = $1;
+        my $events_l = eval( $1."::get_events()") ;
+        foreach my $event_name (@{$events_l}) {
+            $event_hash->{$event_name} = $event_module;
+        }
+
     }
-    &main::daemon_log("open_socket to: $PeerAddr", 7);
-    return $socket;
 }
 
 
@@ -187,104 +267,94 @@ sub open_socket {
 #  DESCRIPTION:  handels the proceeded distribution to the appropriated functions
 #===============================================================================
 sub process_incoming_msg {
-    my ($crypted_msg) = @_ ;
-    if(not defined $crypted_msg) {
-        &main::daemon_log("function 'process_incoming_msg': got no msg", 7);
-    }
-    &main::daemon_log("GosaPackages: incoming msg: \n$crypted_msg", 7);
-
-    $crypted_msg =~ /^([\s\S]*?)\.(\d{1,3}?)\.(\d{1,3}?)\.(\d{1,3}?)\.(\d{1,3}?)$/;
-    $crypted_msg = $1;
-    my $host = sprintf("%s.%s.%s.%s", $2, $3, $4, $5);
-    # collect addresses from possible incoming clients
-    # only gosa is allowd as incoming client
-    &main::daemon_log("GosaPackages: host_key: $host", 7);
-    &main::daemon_log("GosaPackages: key_passwd: $gosa_passwd", 7);
-
-    $gosa_cipher = &create_ciphering($gosa_passwd);
-
-    # determine the correct passwd for deciphering of the incoming msgs
-    my $msg = "";
-    my $msg_hash;
-    eval{
-        $msg = &decrypt_msg($crypted_msg, $gosa_cipher);
-        &main::daemon_log("GosaPackages: decrypted_msg: \n$msg", 7);
-
-        $msg_hash = $xml->XMLin($msg, ForceArray=>1);
-    };
-    if($@) {
-        &main::daemon_log("WARNING: GosaPackages do not understand the message:", 5);
-        &main::daemon_log("$@", 7);
-        return;
-    }
-
+    my ($msg, $msg_hash) = @_ ;
     my $header = @{$msg_hash->{header}}[0];
+    my $out_msg;
     
-    &main::daemon_log("recieve '$header' at GosaPackages from $host", 1);
-    &main::daemon_log("$msg", 7);
+    &main::daemon_log("GosaPackages: receive '$header'", 1);
     
-    my $out_msg;
     if ($header =~ /^job_/) {
         $out_msg = &process_job_msg($msg, $msg_hash);
-    } elsif ($header =~ /^gosa_/) {
-        $out_msg = &process_gosa_msg($msg, $header);
-    } else {
+    } 
+    elsif ($header =~ /^gosa_/) {
+        $out_msg = &process_gosa_msg($msg, $msg_hash);
+    } 
+    else {
         &main::daemon_log("ERROR: $header is not a valid GosaPackage-header, need a 'job_' or a 'gosa_' prefix");
     }
-    
-    if (not defined $out_msg) {
-        return;
-    }
 
-    if ($out_msg =~ /<jobdb_id>(\d*?)<\/jobdb_id>/) {
+    # keep job queue uptodate and save result and status
+    if (defined ($out_msg) && $out_msg =~ /<jobdb_id>(\d*?)<\/jobdb_id>/) {
         my $job_id = $1;
-        my $sql = "UPDATE '$main::job_queue_table_name' SET status='done', result='$out_msg' WHERE id='$job_id'";
+        my $sql = "UPDATE '".$main::job_queue_table_name.
+            "' SET status='done', result='".$out_msg.
+            "' WHERE id='$job_id'";
         my $res = $main::job_db->exec_statement($sql);
-        return;
+    } 
 
-    } else {
-        my $out_cipher = &create_ciphering($gosa_passwd);
-        $out_msg = &encrypt_msg($out_msg, $out_cipher);
-        return $out_msg;
+    my @out_msg_l;
+    if (defined $out_msg){
+        push(@out_msg_l, $out_msg);
     }
-
+    return \@out_msg_l;
 }
 
+
 sub process_gosa_msg {
-    my ($msg, $header) = @_ ;
+    my ($msg, $msg_hash) = @_ ;
     my $out_msg;
+
+    my $header = @{$msg_hash->{'header'}}[0];
     $header =~ s/gosa_//;
-    &main::daemon_log("GosaPackages: got a gosa msg $header", 5);
 
     # decide wether msg is a core function or a event handler
-    if ( $header eq 'query_jobdb') { $out_msg = &query_jobdb }
-    elsif ($header eq 'delete_jobdb_entry') { $out_msg = &delete_jobdb_entry }
-    elsif ($header eq 'clear_jobdb') { $out_msg = &clear_jobdb }
-    elsif ($header eq 'update_status_jobdb_entry' ) { $out_msg = &update_status_jobdb_entry }
-    elsif ($header eq 'update_timestamp_jobdb_entry' ) { $out_msg = &update_timestamp_jobdb_entry }
-    else {
+    if ( $header eq 'query_jobdb') {
+       $out_msg = &query_jobdb
+    } elsif ($header eq 'delete_jobdb_entry') {
+        $out_msg = &delete_jobdb_entry
+    } elsif ($header eq 'clear_jobdb') {
+       $out_msg = &clear_jobdb
+    } elsif ($header eq 'update_status_jobdb_entry' ) {
+       $out_msg = &update_status_jobdb_entry
+    } elsif ($header eq 'count_jobdb' ) {
+        $out_msg = &count_jobdb
+    } elsif ($header eq 'trigger_action_reboot' ) {
+       print STDERR "trigger_action_reboot\n";
+       print STDERR Dumper($msg);
+    } elsif ($header eq 'trigger_action_wake' ) {
+       # Forward messages to all known servers as "trigger_wake"
+       my $in_hash= &transform_msg2hash($msg);
+       my %data = ( 'macAddress'  => \@{$in_hash->{macAddress}} );
+       $out_msg = &send_msg("trigger_wake", $server_address, "KNOWN_SERVER", \%data);
+    } else {
         # msg could not be assigned to core function
-        # fetch all available eventhandler under $server_event_dir
-        opendir (DIR, $server_event_dir) or &main::daemon_log("ERROR cannot open $server_event_dir: $!\n", 1) and return;
-        while (defined (my $file = readdir (DIR))) {
-            if (not $file eq $header) {
-                next;
-            }
-            # try to deliver incoming msg to eventhandler
-            my $cmd = File::Spec->join($server_event_dir, $header)." '$msg'";
-            &main::daemon_log("GosaPackages: execute event_handler $header", 3);
-            &main::daemon_log("GosaPackages: cmd: $cmd", 7);
-
-            $out_msg = "";
-            open(PIPE, "$cmd 2>&1 |");
-            while(<PIPE>) {
-                $out_msg.=$_;
-            }
-            close(PIPE);
-            &main::daemon_log("GosaPackages: answer of cmd: $out_msg", 5);
-            last;
-        }
+        # maybe it is an eventa
+        if( exists $event_hash->{$header} ) {
+            # a event exists with the header as name
+            &main::daemon_log("found event '$header' at event-module '".$event_hash->{$header}."'", 5);
+            no strict 'refs';
+            $out_msg = &{$event_hash->{$header}."::$header"}($msg, $msg_hash);
+         }
+
+#        opendir (DIR, $server_event_dir) or &main::daemon_log("ERROR cannot open $server_event_dir: $!\n", 1) and return;
+#        while (defined (my $file = readdir (DIR))) {
+#            if (not $file eq $header) {
+#                next;
+#            }
+#            # try to deliver incoming msg to eventhandler
+#            my $cmd = File::Spec->join($server_event_dir, $header)." '$msg'";
+#            &main::daemon_log("GosaPackages: execute event_handler $header", 3);
+#            &main::daemon_log("GosaPackages: cmd: $cmd", 8);
+#
+#            $out_msg = "";
+#            open(PIPE, "$cmd 2>&1 |");
+#            while(<PIPE>) {
+#                $out_msg.=$_;
+#            }
+#            close(PIPE);
+#            &main::daemon_log("GosaPackages: answer of cmd: $out_msg", 5);
+#            last;
+#        }
     }
 
     # if delivery not possible raise error and return 
@@ -303,13 +373,13 @@ sub process_job_msg {
 
     my $header = @{$msg_hash->{header}}[0];
     $header =~ s/job_//;
-    &main::daemon_log("GosaPackages: got a job msg $header", 5);
     
     # check wether mac address is already known in known_daemons or known_clients
-    my $target = 'not known until now';
+    my $target = 'none';
 
     # add job to job queue
     my $func_dic = {table=>$main::job_queue_table_name, 
+                    primkey=>'id',
                     timestamp=>@{$msg_hash->{timestamp}}[0],
                     status=>'waiting', 
                     result=>'none',
@@ -324,29 +394,34 @@ sub process_job_msg {
     }
     
     &main::daemon_log("GosaPackages: $header job successfully added to job queue", 3);
-    return "<xml><1>$res</1></xml>";
+    return "<xml><header>answer</header><source>$server_address</source><target>GOSA</target><answer1>$res</answer1></xml>";
 
 }
 
 
 sub db_res_2_xml {
     my ($db_res) = @_ ;
-    my $xml = "<xml>";
 
-    while ( my ($hit, $hash) = each %{ $db_res } ) {
-        $xml .= "<$hit>";
+    my $xml = "<xml><header>answer</header><source>$server_address</source><target>GOSA</target>";
 
+    my $len_db_res= keys %{$db_res};
+
+    for( my $i= 1; $i<= $len_db_res; $i++ ) {
+        $xml .= "\n<answer$i>";
+        my $hash= $db_res->{$i};
         while ( my ($column_name, $column_value) = each %{$hash} ) {
             $xml .= "<$column_name>";
-            my $xml_content = $column_value;
-            if( $column_name eq "xml" ) {
+            my $xml_content;
+            if( $column_name eq "xmlmessage" ) {
                 $xml_content = &encode_base64($column_value);
+            } else {
+                $xml_content = $column_value;
             }
             $xml .= $xml_content;
             $xml .= "</$column_name>"; 
         }
+        $xml .= "</answer$i>";
 
-        $xml .= "</$hit>";
     }
 
     $xml .= "</xml>";
@@ -361,38 +436,50 @@ sub query_jobdb {
     my $msg_hash = &transform_msg2hash($msg);
 
     # prepare query sql statement
-    my @where = @{$msg_hash->{where}};
-    my $where_hash = {table=>$main::job_queue_table_name };
-    foreach my $where_pram (@where) {
-        my $where_val = @{$msg_hash->{$where_pram}}[0];
-        if (defined $where_val) {
-            $where_hash->{$where_pram} = $where_val;
-        }
-    }
-    # execute db query   
-    my $res_hash = $main::job_db->select_dbentry($where_hash);
+    my $select= &get_select_statement($msg, $msg_hash);
+    my $table= $main::job_queue_table_name;
+    my $where= &get_where_statement($msg, $msg_hash);
+    my $limit= &get_limit_statement($msg, $msg_hash);
+    my $orderby= &get_orderby_statement($msg, $msg_hash);
+    my $sql_statement= "SELECT $select FROM $table $where $orderby $limit";
 
+    # execute db query   
+    my $res_hash = $main::job_db->select_dbentry($sql_statement);
     my $out_xml = &db_res_2_xml($res_hash);
+
     return $out_xml;
 }
 
+
+sub count_jobdb {
+    my ($msg)= @_;
+    my $out_xml= "<xml><count>error</count></xml>";
+
+    # prepare query sql statement
+    my $table= $main::job_queue_table_name;
+    my $sql_statement= "SELECT * FROM $table ";
+    
+    # execute db query
+    my $res_hash = $main::job_db->select_dbentry($sql_statement);
+
+    my $count = keys(%{$res_hash});
+    $out_xml= "<xml><header>answer</header><source>$server_address</source><target>GOSA</target><count>$count</count></xml>";
+
+    return $out_xml;
+}
+
+
 sub delete_jobdb_entry {
     my ($msg) = @_ ;
     my $msg_hash = &transform_msg2hash($msg);
     
     # prepare query sql statement
-    my @where = @{$msg_hash->{where}};
-    my $where_hash = {table=>$main::job_queue_table_name };
-    foreach my $where_pram (@where) {
-        my $where_val = @{$msg_hash->{$where_pram}}[0];
-        if (defined $where_val) {
-            $where_hash->{$where_pram} = $where_val;
-        }
-    }
+    my $table= $main::job_queue_table_name;
+    my $where= &get_where_statement($msg, $msg_hash);
+    my $sql_statement = "DELETE FROM $table $where";
     
     # execute db query
-    my $db_res = $main::job_db->del_dbentry($where_hash);
+    my $db_res = $main::job_db->del_dbentry($sql_statement);
 
     my $res;
     if( $db_res > 0 ) { 
@@ -402,99 +489,61 @@ sub delete_jobdb_entry {
     }
 
     # prepare xml answer
-    my $out_xml = "<xml><1>$res</1></xml>";
+    my $out_xml = "<xml><header>answer</header><source>$server_address</source><target>GOSA</target><answer1>$res</answer1></xml>";
     return $out_xml;
 
 }
 
+
 sub clear_jobdb {
     my ($msg) = @_ ;
     my $msg_hash = &transform_msg2hash($msg);
+    my $error= 0;
+    my $out_xml= "<xml><answer1>1</answer1></xml>";
+    my $table= $main::job_queue_table_name;
     
-    my $where_hash = {table=>$main::job_queue_table_name };
+    my $sql_statement = "DELETE FROM $table";
+    my $db_res = $main::job_db->del_dbentry($sql_statement);
+    if( not $db_res > 0 ) { $error++; };
     
-    # execute db query
-    my $db_res = $main::job_db->del_dbentry($where_hash);
-    print STDERR "db_res=$db_res\n";
-    my $res;
-    if( $db_res eq '0E0' ) { 
-        $res = 0 ;
-    } else {
-        $res = 1;
+    if( $error == 0 ) {
+        $out_xml = "<xml><header>answer</header><source>$server_address</source><target>GOSA</target><answer1>0</answer1></xml>";
     }
-
-    # prepare xml answer
-    my $out_xml = "<xml><1>$res</1></xml>";
+   
     return $out_xml;
 }
 
+
 sub update_status_jobdb_entry {
     my ($msg) = @_ ;
     my $msg_hash = &transform_msg2hash($msg);
-    
-    # prepare query sql statement
-    my $update_hash = {table=>$main::job_queue_table_name };
-    if( exists $msg_hash->{where} ) {
-        $update_hash->{where} = $msg_hash->{where};
-    } else {
-        $update_hash->{where} = [];
-    }
-
-    if( not exists $msg_hash->{update}[0]->{status} ) {
-        return "<xml><1>1</1></xml>";
-    }
-    $update_hash->{update} = [ { status=>$msg_hash->{update}[0]->{status} } ];
-    #$update_hash->{update} = $msg_hash->{update};
-
-    # execute db query
-    my $db_res = $main::job_db->update_dbentry($update_hash);
-
-    # transform db answer to error returnment
-    my $res;
-    if( $db_res > 0 ) { 
-        $res = 0 ;
-    } else {
-        $res = 1;
-    }
-
-    # prepare xml answer
-    my $out_xml = "<xml><1>$res</1></xml>";
-    return $out_xml;
-}
+    my $error= 0;
+    my $out_xml= "<xml><header>answer</header><source>$server_address</source><target>GOSA</target><answer1>1</answer1></xml>";
 
-sub update_timestamp_jobdb_entry {
-    my ($msg) = @_ ;
-    my $msg_hash = &transform_msg2hash($msg);
+    my @len_hash = keys %{$msg_hash};
+    if( 0 == @len_hash) {  $error++; };
     
     # prepare query sql statement
-    my $update_hash = {table=>$main::job_queue_table_name };
-    if( exists $msg_hash->{where} ) {
-        $update_hash->{where} = $msg_hash->{where};
-    } else {
-        $update_hash->{where} = [];
-    }
+    if( $error == 0) {
+        my $table= $main::job_queue_table_name;
+        my $where= &get_where_statement($msg, $msg_hash);
+        my $update= &get_update_statement($msg, $msg_hash);
 
-    if( not exists $msg_hash->{update}[0]->{timestamp} ) {
-        return "<xml><1>1</1></xml>";
-    }
+        my $sql_statement = "UPDATE $table $update $where";
 
-    $update_hash->{update} = [ { timestamp=>$msg_hash->{update}[0]->{timestamp} } ];
+        # execute db query
+        my $db_res = $main::job_db->update_dbentry($sql_statement);
 
-    # execute db query
-    my $db_res = $main::job_db->update_dbentry($update_hash);
+        # check success of db update
+        if( not $db_res > 0 ) { $error++; };
+    }
 
-    # transform db answer to error returnment
-    my $res;
-    if( $db_res > 0 ) { 
-        $res = 0 ;
-    } else {
-        $res = 1;
+    if( $error == 0) {
+        $out_xml = "<xml><header>answer</header><source>$server_address</source><target>GOSA</target><answer1>0</answer1></xml>";
     }
 
-    # prepare xml answer
-    my $out_xml = "<xml><1>$res</1></xml>";
     return $out_xml;
-
 }