bug: improve error message in event push

[steebchen]

This should return forbidden only when the query can't find the tenant or the authentication is wrong. Otherwise, the error message should be different and an internal or unavailable error should be returned. This can happen when the database query returns an error (e.g. due to it being unavailable).

hatchet/internal/services/grpc/middleware/auth.go

Lines 37 to 51 in 78685d0

	if err != nil {
	a.l.Debug().Err(err).Msg("error validating tenant token")
	return nil, forbidden
	}
	// get the tenant id
	queriedTenant, err := a.config.Repository.Tenant().GetTenantByID(tenantId)
	if err != nil {
	a.l.Debug().Err(err).Msg("error getting tenant by id")
	return nil, forbidden
	}

An important part is this line

hatchet/internal/auth/token/token.go

Line 145 in 73adb77

if err != nil {

as this needs to be checked for a NotFound error, and then it should return a forbidden error if something is off with the jwt, and in all other cases a generic error with a 500 status code