s3: Support S3 compatible services like Google Storage #221
Conversation
|
related to work i'm also trying to do here: advancedtelematic/treehub#75 |
Services like Google Storage expose s3 compatible APIs that can accessed with a non-aws URL. Signed-off-by: Andy Doan <[email protected]>
|
Thanks for the pr. Can you run |
| .withRegion(credentials.region) | ||
| .build() | ||
| protected lazy val s3client = { | ||
| if(credentials.endpointUrl.length() > 0) { |
There was a problem hiding this comment.
this could be:
protected lazy val s3client = {
val builder = AmazonS3ClientBuilder.standard().withCredentials(credentials)
if(credentials.endpointUrl.nonEmpty)
builder.withEndpointConfiguration(new AwsClientBuilder.EndpointConfiguration(credentials.endpointUrl, credentials.region.getName))
else
builder
}.build()
There was a problem hiding this comment.
else
builder.withRegion(credentials.region)
Oder?
Signed-off-by: Andy Doan <[email protected]>
|
I think this last push addresses the comments. I'm barely capable of writing Scala, so this is just the minimal way I can find to make it work |
|
ping. |
| } | ||
|
|
||
| test("uploading a target changes targets json with custom endpoint url") { | ||
| lazy val credentials = new S3Credentials("", "", "", Regions.fromName("us-central-1"), "https://storage.googleapis.com") |
There was a problem hiding this comment.
This test runs with my credentials, which are just aws credentials, not google, something wrong here?
| bucketId = ${?TUF_REPOSERVER_AWS_BUCKET_ID} | ||
| region = "eu-central-1" | ||
| region = ${?TUF_REPOSERVER_AWS_REGION} | ||
| endpointUrl = ${?TUF_REPOSERVER_S3_URL} |
There was a problem hiding this comment.
this needs an extra line:
endpointUrl = ""
Otherwise wont run without TUF_REPOSERVER_S3_URL set.
|
@doanac If you still want to merge this please open a PR at https://github.com/uptane/ota-tuf We can get this merged soon. Thanks |
3 participants
Services like Google Storage expose s3 compatible APIs that can
accessed with a non-aws URL.
Signed-off-by: Andy Doan [email protected]