ipng/s3-genindex
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

go fmt

Browse Source
This commit is contained in:
Pim van Pelt
2025-12-03 00:17:43 +01:00
parent 45ee30fae1
commit d1db4d5e68
2 changed files with 23 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
cmd/s3-genindex/main.go
View File

@@ -64,7 +64,7 @@ func processS3Bucket(s3Config *S3Config, opts *indexgen.Options) error {
// Get credentials from environment variables
accessKey := os.Getenv("AWS_ACCESS_KEY_ID")
secretKey := os.Getenv("AWS_SECRET_ACCESS_KEY")
if accessKey == "" || secretKey == "" {
return fmt.Errorf("AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY environment variables must be set")
}
@@ -77,8 +77,8 @@ func processS3Bucket(s3Config *S3Config, opts *indexgen.Options) error {
// Create S3 client with custom endpoint resolver
client := s3.NewFromConfig(cfg, func(o *s3.Options) {
o.BaseEndpoint = aws.String(fmt.Sprintf("%s://%s",
map[bool]string{true: "https", false: "http"}[s3Config.UseSSL],
o.BaseEndpoint = aws.String(fmt.Sprintf("%s://%s",
map[bool]string{true: "https", false: "http"}[s3Config.UseSSL],
s3Config.Endpoint))
o.UsePathStyle = true // Use path-style URLs for MinIO compatibility
})
@@ -126,15 +126,15 @@ func processS3Bucket(s3Config *S3Config, opts *indexgen.Options) error {
}
entry := indexgen.FileEntry{
Name: keyName,
Path: keyName,
IsDir: false,
Size: *obj.Size,
ModTime: *obj.LastModified,
IsSymlink: false,
IconType: indexgen.GetIconType(keyName),
SizePretty: indexgen.PrettySize(*obj.Size),
ModTimeISO: obj.LastModified.Format("2006-01-02T15:04:05Z"),
Name: keyName,
Path: keyName,
IsDir: false,
Size: *obj.Size,
ModTime: *obj.LastModified,
IsSymlink: false,
IconType: indexgen.GetIconType(keyName),
SizePretty: indexgen.PrettySize(*obj.Size),
ModTimeISO: obj.LastModified.Format("2006-01-02T15:04:05Z"),
}
// Set CSS class based on file type

22
cmd/s3-genindex/main_test.go
View File

@@ -150,28 +150,28 @@ func TestS3FlagHandling(t *testing.T) {
func TestMutualExclusionAndRequiredFlags(t *testing.T) {
tests := []struct {
name string
args []string
name string
args []string
shouldExitWithError bool
}{
{
name: "no flags provided",
args: []string{"s3-genindex"},
name: "no flags provided",
args: []string{"s3-genindex"},
shouldExitWithError: true,
},
{
name: "directory flag only",
args: []string{"s3-genindex", "-d", "/tmp"},
name: "directory flag only",
args: []string{"s3-genindex", "-d", "/tmp"},
shouldExitWithError: false,
},
{
name: "s3 flag only",
args: []string{"s3-genindex", "-s3", "http://example.com/bucket"},
name: "s3 flag only",
args: []string{"s3-genindex", "-s3", "http://example.com/bucket"},
shouldExitWithError: false,
},
{
name: "both flags provided (mutual exclusion)",
args: []string{"s3-genindex", "-d", "/tmp", "-s3", "http://example.com/bucket"},
name: "both flags provided (mutual exclusion)",
args: []string{"s3-genindex", "-d", "/tmp", "-s3", "http://example.com/bucket"},
shouldExitWithError: true,
},
}
@@ -257,7 +257,7 @@ func TestParseS3URL(t *testing.T) {
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
result, err := parseS3URL(tt.url)
if tt.hasError {
if err == nil {
t.Errorf("parseS3URL(%q) expected error, got nil", tt.url)