fix immutable columns for session_state and session_connnection_state (#449)

internal/db/migrations/postgres.gen.go

@@ -3764,7 +3764,8 @@ begin;
     immutable_columns
   before
   update on session_state
-    for each row execute procedure immutable_columns('session_id', 'start_time', 'previous_end_time');
+    for each row execute procedure immutable_columns('session_id', 'state', 'start_time', 'previous_end_time');
+
     
   create or replace function
     insert_session_state()
@@ -4101,7 +4102,7 @@ begin;
     immutable_columns
   before
   update on session_connection_state
-    for each row execute procedure immutable_columns('connection_id', 'state', 'start_time');
+    for each row execute procedure immutable_columns('connection_id', 'state', 'start_time', 'previous_end_time');
 
   create or replace function
     insert_session_connection_state()

internal/db/migrations/postgres/50_session.up.sql

@@ -408,7 +408,8 @@ begin;
     immutable_columns
   before
   update on session_state
-    for each row execute procedure immutable_columns('session_id', 'start_time', 'previous_end_time');
+    for each row execute procedure immutable_columns('session_id', 'state', 'start_time', 'previous_end_time');
+
     
   create or replace function
     insert_session_state()

internal/db/migrations/postgres/51_connection.up.sql

@@ -249,7 +249,7 @@ begin;
     immutable_columns
   before
   update on session_connection_state
-    for each row execute procedure immutable_columns('connection_id', 'state', 'start_time');
+    for each row execute procedure immutable_columns('connection_id', 'state', 'start_time', 'previous_end_time');
 
   create or replace function
     insert_session_connection_state()

internal/session/immutable_fields_test.go

@@ -119,6 +119,15 @@ func TestState_ImmutableFields(t *testing.T) {
 			}(),
 			fieldMask: []string{"PublicId"},
 		},
+		{
+			name: "status",
+			update: func() *State {
+				s := new.Clone().(*State)
+				s.Status = "active"
+				return s
+			}(),
+			fieldMask: []string{"Status"},
+		},
 		{
 			name: "start time",
 			update: func() *State {
@@ -258,6 +267,15 @@ func TestConnectionState_ImmutableFields(t *testing.T) {
 			}(),
 			fieldMask: []string{"PublicId"},
 		},
+		{
+			name: "status",
+			update: func() *ConnectionState {
+				s := new.Clone().(*ConnectionState)
+				s.Status = "closed"
+				return s
+			}(),
+			fieldMask: []string{"Status"},
+		},
 		{
 			name: "start time",
 			update: func() *ConnectionState {