should we touch timestamps when internally updating?

servers/slapd/back-monitor/conn.c

@@ -217,6 +217,8 @@ monitor_subsys_conn_update(
 		}
 		a->a_vals[ 0 ].bv_len = len;
 		AC_MEMCPY( a->a_vals[ 0 ].bv_val, buf, len + 1 );
+
+		/* FIXME: touch modifyTimestamp? */
 	}
 
 	return( 0 );

servers/slapd/back-monitor/log.c

@@ -148,7 +148,9 @@ monitor_subsys_log_modify(
 		Modification	*mod = &ml->sml_mod;
 
 		/*
-		 * accept all operational attributes
+		 * accept all operational attributes;
+		 * this includes modifersName and modifyTimestamp
+		 * if lastmod is "on"
 		 */
 		if ( is_at_operational( mod->sm_desc->ad_type ) ) {
 			( void ) attr_delete( &e->e_attrs, mod->sm_desc );

servers/slapd/back-monitor/operation.c

@@ -215,6 +215,8 @@ monitor_subsys_ops_update(
 	UI2BV( &a->a_vals[ 0 ], nCompleted );
 	ldap_pvt_mp_clear( nCompleted );
 
+	/* FIXME: touch modifyTimestamp? */
+
 	return( 0 );
 }
 

servers/slapd/back-monitor/rww.c

@@ -217,6 +217,8 @@ monitor_subsys_rww_update(
 	AC_MEMCPY( a->a_vals[ 0 ].bv_val, buf, len + 1 );
 	a->a_vals[ 0 ].bv_len = len;
 
+	/* FIXME: touch modifyTimestamp? */
+
 	return( 0 );
 }
 

servers/slapd/back-monitor/sent.c

@@ -201,6 +201,8 @@ monitor_subsys_sent_update(
 	UI2BV( &a->a_vals[ 0 ], n );
 	ldap_pvt_mp_clear( n );
 
+	/* FIXME: touch modifyTimestamp? */
+
 	return 0;
 }
 

servers/slapd/back-monitor/thread.c

@@ -197,6 +197,8 @@ monitor_subsys_thread_update(
 	a->a_vals[ 0 ].bv_len = len;
 	AC_MEMCPY( a->a_vals[ 0 ].bv_val, buf, len + 1 );
 
+	/* FIXME: touch modifyTimestamp? */
+
 	return( 0 );
 }
 

servers/slapd/back-monitor/time.c

@@ -224,6 +224,8 @@ monitor_subsys_time_update(
 
 		assert( len == a->a_vals[ 0 ].bv_len );
 		AC_MEMCPY( a->a_vals[ 0 ].bv_val, tmbuf, len );
+
+		/* FIXME: touch modifyTimestamp? */
 	}
 
 	return( 0 );