Return 404 from GET Bindings for Expired Bindings

internal/broker/bind_get.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"net/http"
+	"time"
 
 	"github.com/kyma-project/kyma-environment-broker/internal/storage"
 	"github.com/pivotal-cf/brokerapi/v8/domain"
@@ -34,6 +35,11 @@ func (b *GetBindingEndpoint) GetBinding(_ context.Context, instanceID, bindingID
 		return domain.GetBindingSpec{}, apiresponses.NewFailureResponse(fmt.Errorf(message), http.StatusNotFound, message)
 	}
 
+	if binding.ExpiresAt.Before(time.Now()) {
+		message := "Binding expired"
+		return domain.GetBindingSpec{}, apiresponses.NewFailureResponse(fmt.Errorf(message), http.StatusNotFound, message)
+	}
+
 	if err != nil {
 		b.log.Errorf("GetBinding error: %s", err)
 		message := fmt.Sprintf("Unexpected error: %s", err)

internal/broker/bind_get_test.go

@@ -0,0 +1,44 @@
+package broker
+
+import (
+	"context"
+	"net/http"
+	"testing"
+	"time"
+
+	"github.com/kyma-project/kyma-environment-broker/internal"
+	mocks "github.com/kyma-project/kyma-environment-broker/internal/storage/automock"
+	"github.com/pivotal-cf/brokerapi/v8/domain"
+	"github.com/pivotal-cf/brokerapi/v8/domain/apiresponses"
+	"github.com/sirupsen/logrus"
+	"github.com/stretchr/testify/require"
+)
+
+func TestGetBinding(t *testing.T) {
+
+	t.Run("should return 404 code for the expired binding", func(t *testing.T) {
+		// given
+		mockBindings := new(mocks.Bindings)
+
+		expiredBinding := &internal.Binding{
+			ExpiresAt: time.Now().Add(-1 * time.Hour),
+		}
+
+		mockBindings.On("Get", "test-instance-id", "test-binding-id").Return(expiredBinding, nil)
+
+		endpoint := &GetBindingEndpoint{
+			bindings: mockBindings,
+			log:      &logrus.Logger{}, // Assuming you have a mock logger
+		}
+
+		// when
+		_, err := endpoint.GetBinding(context.Background(), "test-instance-id", "test-binding-id", domain.FetchBindingDetails{})
+
+		// then
+		require.NotNil(t, err)
+		apiErr, ok := err.(*apiresponses.FailureResponse)
+		require.True(t, ok)
+		require.Equal(t, http.StatusNotFound, apiErr.ValidatedStatusCode(nil))
+		mockBindings.AssertExpectations(t)
+	})
+}

internal/storage/ext.go

@@ -128,6 +128,7 @@ type SubaccountStates interface {
 	ListStates() ([]internal.SubaccountState, error)
 }
 
+//go:generate mockery --name=Bindings --output=automock --case=underscore
 type Bindings interface {
 	Insert(binding *internal.Binding) error
 	Get(instanceID string, bindingID string) (*internal.Binding, error)