Skip to content

Commit

Permalink
Merge pull request #941 from CurtisEppel/fix-acl-type-discrepancy
Browse files Browse the repository at this point in the history
Fix acl type declarations to match implementation
  • Loading branch information
Nevon committed Nov 4, 2020
2 parents 7154895 + 9b93ada commit 668b85e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions types/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -370,7 +370,7 @@ export interface AclResource {
export type AclEntry = Acl & AclResource

export type DescribeAclResource = AclResource & {
acls: Acl[]
acl: Acl[]
}

export interface DescribeAclResponse {
Expand Down Expand Up @@ -454,7 +454,7 @@ export type Admin = {
describeGroups(groupIds: string[]): Promise<GroupDescriptions>
describeAcls(options: AclFilter): Promise<DescribeAclResponse>
deleteAcls(options: { filters: AclFilter[] }): Promise<DeleteAclResponse>
createAcls(options: { acls: AclEntry[] }): Promise<boolean>
createAcls(options: { acl: AclEntry[] }): Promise<boolean>
deleteTopicRecords(options: { topic: string; partitions: SeekEntry[] }): Promise<void>
logger(): Logger
on(
Expand Down
4 changes: 2 additions & 2 deletions types/tests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -220,12 +220,12 @@ const runAdmin = async () => {
resourceType: r.resourceType,
resourceName: r.resourceName,
resourcePatternType: r.resourcePatternType,
acls: r.acls,
acls: r.acl,
})),
})

const createAcls = await admin.createAcls({
acls: [
acl: [
{
resourceType: AclResourceTypes.TOPIC,
resourceName: 'topic-name',
Expand Down

0 comments on commit 668b85e

Please sign in to comment.