ga4gh / ga4gh/htsget-refserver

Consider improving S3 creds mgmt

Open
#36 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
12
Forks
4
PR merge metrics
No merged PRs in 30d

Description

The following code is present on two different locations of the code:

```diff
- defaultCfg, err := config.LoadDefaultConfig(context.TODO())
+ defaultCfg, err := config.LoadDefaultConfig(
+ context.Background(),
+ )
```

The additions represent the change needed to support AWS CLI v2 properly (default in 2023?). Otherwise all `HeadObject` requests will fail with 400.

I'd open a PR, but seems like this project is abandoned by looking at the outstanding issues and PRs from external collaborators?

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the codebase for the two occurrences of config.LoadDefaultConfig and inspect how they feed HeadObject requests. Apply the context change shown in the issue at both locations, then verify that HeadObject requests work with AWS CLI v2 credentials.

Written by the indexing model from the issue text.

Assessment

Tech stack
aws, go
Domain
backend, cloud
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.