Skip to content

Commit 0f9338d

Browse files
committed
fix perms for creating cred set
1 parent fc88aff commit 0f9338d

File tree

4 files changed

+13
-12
lines changed

4 files changed

+13
-12
lines changed

apps/sim/app/api/credential-sets/[id]/invite/route.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,8 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
7878
return NextResponse.json({ error: 'Credential set not found' }, { status: 404 })
7979
}
8080

81-
if (result.role !== 'admin') {
82-
return NextResponse.json({ error: 'Admin permissions required' }, { status: 403 })
81+
if (result.role !== 'admin' && result.role !== 'owner') {
82+
return NextResponse.json({ error: 'Admin or owner permissions required' }, { status: 403 })
8383
}
8484

8585
const body = await req.json()
@@ -193,8 +193,8 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
193193
return NextResponse.json({ error: 'Credential set not found' }, { status: 404 })
194194
}
195195

196-
if (result.role !== 'admin') {
197-
return NextResponse.json({ error: 'Admin permissions required' }, { status: 403 })
196+
if (result.role !== 'admin' && result.role !== 'owner') {
197+
return NextResponse.json({ error: 'Admin or owner permissions required' }, { status: 403 })
198198
}
199199

200200
await db

apps/sim/app/api/credential-sets/[id]/members/route.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,8 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
125125
return NextResponse.json({ error: 'Credential set not found' }, { status: 404 })
126126
}
127127

128-
if (result.role !== 'admin') {
129-
return NextResponse.json({ error: 'Admin permissions required' }, { status: 403 })
128+
if (result.role !== 'admin' && result.role !== 'owner') {
129+
return NextResponse.json({ error: 'Admin or owner permissions required' }, { status: 403 })
130130
}
131131

132132
const [memberToRemove] = await db

apps/sim/app/api/credential-sets/[id]/route.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,8 @@ export async function PUT(req: NextRequest, { params }: { params: Promise<{ id:
7575
return NextResponse.json({ error: 'Credential set not found' }, { status: 404 })
7676
}
7777

78-
if (result.role !== 'admin') {
79-
return NextResponse.json({ error: 'Admin permissions required' }, { status: 403 })
78+
if (result.role !== 'admin' && result.role !== 'owner') {
79+
return NextResponse.json({ error: 'Admin or owner permissions required' }, { status: 403 })
8080
}
8181

8282
const body = await req.json()
@@ -138,8 +138,8 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
138138
return NextResponse.json({ error: 'Credential set not found' }, { status: 404 })
139139
}
140140

141-
if (result.role !== 'admin') {
142-
return NextResponse.json({ error: 'Admin permissions required' }, { status: 403 })
141+
if (result.role !== 'admin' && result.role !== 'owner') {
142+
return NextResponse.json({ error: 'Admin or owner permissions required' }, { status: 403 })
143143
}
144144

145145
await db.delete(credentialSetMember).where(eq(credentialSetMember.credentialSetId, id))

apps/sim/app/api/credential-sets/route.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,10 @@ export async function POST(req: Request) {
9595
.where(and(eq(member.userId, session.user.id), eq(member.organizationId, organizationId)))
9696
.limit(1)
9797

98-
if (membership.length === 0 || membership[0].role !== 'admin') {
98+
const role = membership[0]?.role
99+
if (membership.length === 0 || (role !== 'admin' && role !== 'owner')) {
99100
return NextResponse.json(
100-
{ error: 'Admin permissions required to create credential sets' },
101+
{ error: 'Admin or owner permissions required to create credential sets' },
101102
{ status: 403 }
102103
)
103104
}

0 commit comments

Comments
 (0)