Skip to content

Commit

Permalink
fix(fs-aws): correct typing for credentials
Browse files Browse the repository at this point in the history
  • Loading branch information
blacha committed Aug 23, 2023
1 parent b3cc7f5 commit 586cccd
Showing 1 changed file with 3 additions and 11 deletions.
14 changes: 3 additions & 11 deletions packages/fs-aws/src/fs.s3.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,19 +7,11 @@ import {
S3Client,
} from '@aws-sdk/client-s3';
import { Upload } from '@aws-sdk/lib-storage';
import {
FileInfo,
FileSystem,
FileSystemAction,
FileSystemProvider,
FsError,
ListOptions,
WriteOptions,
isRecord,
} from '@chunkd/fs';
import { FileInfo, FileSystem, FileSystemAction, FsError, ListOptions, WriteOptions, isRecord } from '@chunkd/fs';
import { SourceAwsS3 } from '@chunkd/source-aws';
import type { Readable } from 'node:stream';
import { PassThrough } from 'node:stream';
import { AwsS3CredentialProvider } from './credentials.js';

function isReadable(r: any): r is Readable {
return r != null && typeof r['read'] === 'function';
Expand Down Expand Up @@ -51,7 +43,7 @@ export class FsAwsS3 implements FileSystem {
static WriteTestSuffix = '.fsa-test';

/** Attempt to lookup credentials when permission failures happen */
credentials?: FileSystemProvider<FsAwsS3>;
credentials?: AwsS3CredentialProvider;

/** Buckets we have already tested writing too and should skip testing multiple times */
writeTests = new Set<string>();
Expand Down

0 comments on commit 586cccd

Please sign in to comment.