mirror of
https://github.com/hanzoai/livekit.git
synced 2026-10-30 18:29:34 +00:00
zeekay
e9292da479
This fork lived at github.com/hanzoai/livekit but still declared
`module github.com/livekit/livekit-server`, so all 505 self-references pointed
at upstream. A fork that does not own its module path is the exact thing the
owned-module-paths rule exists to prevent: every import line in every consumer
carries someone else's brand, and `go get` resolves to their code, not ours.
module + 505 self-imports github.com/livekit/livekit-server -> github.com/hanzoai/livekit
KV client github.com/redis/go-redis/v9 -> github.com/hanzokv/go/v9
message bus github.com/livekit/psrpc -> github.com/hanzoai/psrpc
psrpc had to be forked (hanzoai/psrpc, Apache-2.0) because its bus is typed
against the KV client — the same shape redsync had for hanzoai/git.
DELIBERATELY NOT FORKED: github.com/livekit/{protocol,mediatransportutil,
mageutil} keep their addresses. Only `protocol` still reaches go-redis, through
a single file (redis/redis.go) out of 305. Forking an 11.5MB core wire-protocol
library that this server must track in lockstep — and that upstream pushed
yesterday — to change one file trades a tidy dependency graph for permanent
divergence on the fastest-moving dependency here. So one indirect
github.com/redis/go-redis entry remains, via livekit/protocol/redis, and that
is the correct place for it to live.
139 lines
3.9 KiB
Go
139 lines
3.9 KiB
Go
// Copyright 2023 LiveKit, Inc.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package service_test
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
"github.com/twitchtv/twirp"
|
|
|
|
"github.com/livekit/protocol/auth"
|
|
"github.com/livekit/protocol/livekit"
|
|
"github.com/livekit/protocol/rpc"
|
|
"github.com/livekit/protocol/rpc/rpcfakes"
|
|
|
|
"github.com/hanzoai/livekit/pkg/config"
|
|
"github.com/hanzoai/livekit/pkg/routing/routingfakes"
|
|
"github.com/hanzoai/livekit/pkg/service"
|
|
"github.com/hanzoai/livekit/pkg/service/servicefakes"
|
|
)
|
|
|
|
func TestDeleteRoom(t *testing.T) {
|
|
t.Run("missing permissions", func(t *testing.T) {
|
|
svc := newTestRoomService(config.LimitConfig{})
|
|
grant := &auth.ClaimGrants{
|
|
Video: &auth.VideoGrant{},
|
|
}
|
|
ctx := service.WithGrants(context.Background(), grant, "")
|
|
_, err := svc.DeleteRoom(ctx, &livekit.DeleteRoomRequest{
|
|
Room: "testroom",
|
|
})
|
|
require.Error(t, err)
|
|
})
|
|
}
|
|
|
|
func TestMetaDataLimits(t *testing.T) {
|
|
t.Run("metadata exceed limits", func(t *testing.T) {
|
|
svc := newTestRoomService(config.LimitConfig{MaxMetadataSize: 5})
|
|
grant := &auth.ClaimGrants{
|
|
Video: &auth.VideoGrant{},
|
|
}
|
|
ctx := service.WithGrants(context.Background(), grant, "")
|
|
_, err := svc.UpdateParticipant(ctx, &livekit.UpdateParticipantRequest{
|
|
Room: "testroom",
|
|
Identity: "123",
|
|
Metadata: "abcdefg",
|
|
})
|
|
terr, ok := err.(twirp.Error)
|
|
require.True(t, ok)
|
|
require.Equal(t, twirp.InvalidArgument, terr.Code())
|
|
|
|
_, err = svc.UpdateRoomMetadata(ctx, &livekit.UpdateRoomMetadataRequest{
|
|
Room: "testroom",
|
|
Metadata: "abcdefg",
|
|
})
|
|
terr, ok = err.(twirp.Error)
|
|
require.True(t, ok)
|
|
require.Equal(t, twirp.InvalidArgument, terr.Code())
|
|
})
|
|
|
|
notExceedsLimitsSvc := map[string]*TestRoomService{
|
|
"metadata exceeds limits": newTestRoomService(config.LimitConfig{MaxMetadataSize: 5}),
|
|
"metadata no limits": newTestRoomService(config.LimitConfig{}), // no limits
|
|
}
|
|
|
|
for n, s := range notExceedsLimitsSvc {
|
|
svc := s
|
|
t.Run(n, func(t *testing.T) {
|
|
grant := &auth.ClaimGrants{
|
|
Video: &auth.VideoGrant{},
|
|
}
|
|
ctx := service.WithGrants(context.Background(), grant, "")
|
|
_, err := svc.UpdateParticipant(ctx, &livekit.UpdateParticipantRequest{
|
|
Room: "testroom",
|
|
Identity: "123",
|
|
Metadata: "abc",
|
|
})
|
|
terr, ok := err.(twirp.Error)
|
|
require.True(t, ok)
|
|
require.NotEqual(t, twirp.InvalidArgument, terr.Code())
|
|
|
|
_, err = svc.UpdateRoomMetadata(ctx, &livekit.UpdateRoomMetadataRequest{
|
|
Room: "testroom",
|
|
Metadata: "abc",
|
|
})
|
|
terr, ok = err.(twirp.Error)
|
|
require.True(t, ok)
|
|
require.NotEqual(t, twirp.InvalidArgument, terr.Code())
|
|
})
|
|
|
|
}
|
|
}
|
|
|
|
func newTestRoomService(limitConf config.LimitConfig) *TestRoomService {
|
|
router := &routingfakes.FakeRouter{}
|
|
allocator := &servicefakes.FakeRoomAllocator{}
|
|
store := &servicefakes.FakeServiceStore{}
|
|
svc, err := service.NewRoomService(
|
|
limitConf,
|
|
config.APIConfig{ExecutionTimeout: 2},
|
|
router,
|
|
allocator,
|
|
store,
|
|
nil,
|
|
rpc.NewTopicFormatter(),
|
|
&rpcfakes.FakeTypedRoomClient{},
|
|
&rpcfakes.FakeTypedParticipantClient{},
|
|
)
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
return &TestRoomService{
|
|
RoomService: *svc,
|
|
router: router,
|
|
allocator: allocator,
|
|
store: store,
|
|
}
|
|
}
|
|
|
|
type TestRoomService struct {
|
|
service.RoomService
|
|
router *routingfakes.FakeRouter
|
|
allocator *servicefakes.FakeRoomAllocator
|
|
store *servicefakes.FakeServiceStore
|
|
}
|