[GitHub] [jclouds] jiffysrc edited a comment on pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-27 Thread GitBox
jiffysrc edited a comment on pull request #125: URL: https://github.com/apache/jclouds/pull/125#issuecomment-1001529498 Hi @gaul We will divide the changes in this pr into 2 new separate PR's having, 1 : bucket configuration changes , 2 : Changing to RequestAuthorizeSignatureV4

[GitHub] [jclouds] jiffysrc commented on a change in pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-27 Thread GitBox
jiffysrc commented on a change in pull request #125: URL: https://github.com/apache/jclouds/pull/125#discussion_r775464229 ## File path: apis/s3/src/main/java/org/jclouds/s3/domain/internal/ListVersionsResponseImpl.java ## @@ -0,0 +1,208 @@ +/* + * Licensed to the Apache Softw

[GitHub] [jclouds] jiffysrc commented on a change in pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-27 Thread GitBox
jiffysrc commented on a change in pull request #125: URL: https://github.com/apache/jclouds/pull/125#discussion_r775464079 ## File path: apis/s3/src/main/java/org/jclouds/s3/S3Client.java ## @@ -260,7 +264,7 @@ DeleteResult deleteObjects(@Bucket @EndpointParam(parser = AssignC

[GitHub] [jclouds] jiffysrc commented on a change in pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-27 Thread GitBox
jiffysrc commented on a change in pull request #125: URL: https://github.com/apache/jclouds/pull/125#discussion_r775463338 ## File path: blobstore/src/main/java/org/jclouds/blobstore/options/ListContainerOptions.java ## @@ -231,6 +244,15 @@ public ListContainerOptions delimite

[GitHub] [jclouds] jiffysrc commented on a change in pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-27 Thread GitBox
jiffysrc commented on a change in pull request #125: URL: https://github.com/apache/jclouds/pull/125#discussion_r775462999 ## File path: apis/s3/src/main/java/org/jclouds/s3/S3Client.java ## @@ -640,6 +651,30 @@ void enableBucketLogging(@Bucket @EndpointParam(parser = AssignCo

[GitHub] [jclouds] jiffysrc commented on a change in pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-27 Thread GitBox
jiffysrc commented on a change in pull request #125: URL: https://github.com/apache/jclouds/pull/125#discussion_r775462468 ## File path: apis/s3/src/main/java/org/jclouds/s3/config/S3HttpApiModule.java ## @@ -184,7 +184,7 @@ protected void bindErrorHandlers() { } prot

[GitHub] [jclouds] gaul commented on pull request #128: Update ClientCredentialsClaims.java

2021-12-26 Thread GitBox
gaul commented on pull request #128: URL: https://github.com/apache/jclouds/pull/128#issuecomment-1001393292 Thank you for your contribution @SATYANAN-ANAND! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [jclouds] gaul merged pull request #128: Update ClientCredentialsClaims.java

2021-12-26 Thread GitBox
gaul merged pull request #128: URL: https://github.com/apache/jclouds/pull/128 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] SATYANAN-ANAND commented on a change in pull request #128: Update ClientCredentialsClaims.java

2021-12-26 Thread GitBox
SATYANAN-ANAND commented on a change in pull request #128: URL: https://github.com/apache/jclouds/pull/128#discussion_r775366148 ## File path: apis/oauth/src/main/java/org/jclouds/oauth/v2/domain/ClientCredentialsClaims.java ## @@ -49,6 +50,7 @@ public abstract long nbf()

[GitHub] [jclouds] gaul commented on a change in pull request #128: Update ClientCredentialsClaims.java

2021-12-24 Thread GitBox
gaul commented on a change in pull request #128: URL: https://github.com/apache/jclouds/pull/128#discussion_r775010584 ## File path: apis/oauth/src/main/java/org/jclouds/oauth/v2/domain/ClientCredentialsClaims.java ## @@ -49,6 +50,7 @@ public abstract long nbf(); /

[GitHub] [jclouds] SATYANAN-ANAND opened a new pull request #128: Update ClientCredentialsClaims.java

2021-12-24 Thread GitBox
SATYANAN-ANAND opened a new pull request #128: URL: https://github.com/apache/jclouds/pull/128 JCLOUDS-1591: OAuth: ClientCredentialsJWTBearerTokenFlow.filter method throws Null Pointer Exception -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [jclouds] asfgit merged pull request #127: JCLOUDS-1590: Promote glacier to core

2021-12-17 Thread GitBox
asfgit merged pull request #127: URL: https://github.com/apache/jclouds/pull/127 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-uns

[GitHub] [jclouds] gaul commented on a change in pull request #127: JCLOUDS-1590: Promote glacier to core

2021-12-17 Thread GitBox
gaul commented on a change in pull request #127: URL: https://github.com/apache/jclouds/pull/127#discussion_r771469583 ## File path: apis/glacier/pom.xml ## @@ -0,0 +1,249 @@ + + +http://www.w3.org/2001/XMLSchema-instance"; xmlns="http://maven.apache.org/POM/4.0.0"; +

[GitHub] [jclouds] nacx commented on a change in pull request #127: JCLOUDS-1590: Promote glacier to core

2021-12-17 Thread GitBox
nacx commented on a change in pull request #127: URL: https://github.com/apache/jclouds/pull/127#discussion_r771314072 ## File path: apis/glacier/pom.xml ## @@ -0,0 +1,249 @@ + + +http://www.w3.org/2001/XMLSchema-instance"; xmlns="http://maven.apache.org/POM/4.0.0"; +

[GitHub] [jclouds] gaul commented on a change in pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-12-17 Thread GitBox
gaul commented on a change in pull request #125: URL: https://github.com/apache/jclouds/pull/125#discussion_r746509476 ## File path: apis/s3/src/main/java/org/jclouds/s3/domain/internal/ListVersionsResponseImpl.java ## @@ -0,0 +1,208 @@ +/* + * Licensed to the Apache Software

[GitHub] [jclouds] gaul merged pull request #126: JCLOUDS-1589: Upgrade to log4j 2.16.0

2021-12-16 Thread GitBox
gaul merged pull request #126: URL: https://github.com/apache/jclouds/pull/126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] nacx commented on pull request #126: JCLOUDS-1589: Upgrade to log4j 2.16.0

2021-12-16 Thread GitBox
nacx commented on pull request #126: URL: https://github.com/apache/jclouds/pull/126#issuecomment-995891245 Thanks! :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

[GitHub] [jclouds] gaul opened a new pull request #126: JCLOUDS-1589: Upgrade to log4j 2.16.0

2021-12-16 Thread GitBox
gaul opened a new pull request #126: URL: https://github.com/apache/jclouds/pull/126 This addresses a critical CVE: https://logging.apache.org/log4j/2.x/security.html -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [jclouds] gaul commented on pull request #124: Bump google gson to 2.8.9 due to detected vulnerability

2021-11-10 Thread GitBox
gaul commented on pull request #124: URL: https://github.com/apache/jclouds/pull/124#issuecomment-965013148 Thank you for your contribution @jcabrerizo! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [jclouds] gaul merged pull request #124: Bump google gson to 2.8.9 due to detected vulnerability

2021-11-10 Thread GitBox
gaul merged pull request #124: URL: https://github.com/apache/jclouds/pull/124 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] jcabrerizo commented on pull request #124: Bump google gson to 2.8.9 due to detected vulnerability

2021-11-10 Thread GitBox
jcabrerizo commented on pull request #124: URL: https://github.com/apache/jclouds/pull/124#issuecomment-964873471 Hi @gaul I created the issue: https://issues.apache.org/jira/browse/JCLOUDS-1588 -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [jclouds] jiffysrc opened a new pull request #125: JCLOUDS-1587 S3 Blobstore: Support Bucket Configuration Options - Versioning, Lifecycle, Encryption

2021-11-09 Thread GitBox
jiffysrc opened a new pull request #125: URL: https://github.com/apache/jclouds/pull/125 https://issues.apache.org/jira/browse/JCLOUDS-1587 Support following features in jclouds s3 blobstore Configure the bucket for versioning and get the versioning state of a bucket Confiur

[GitHub] [jclouds] gaul commented on pull request #124: Bump google gson to 2.8.9 due to detected vulnerability

2021-11-09 Thread GitBox
gaul commented on pull request #124: URL: https://github.com/apache/jclouds/pull/124#issuecomment-964657670 Could you open a JIRA issue and tag this commit in the summary? This will ensure that the release notes include it. -- This is an automated message from the Apache Git Service. To

[GitHub] [jclouds] gaul merged pull request #123: JCLOUDS-1586: Upgrade to Guice 5.0.1

2021-11-09 Thread GitBox
gaul merged pull request #123: URL: https://github.com/apache/jclouds/pull/123 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] jcabrerizo opened a new pull request #124: Bump google gson to 2.8.9 due to detected vulnerability

2021-11-09 Thread GitBox
jcabrerizo opened a new pull request #124: URL: https://github.com/apache/jclouds/pull/124 Snyk identifies now previos version of gson as vulnerable. This updates gson to the fixed version skyn report: https://security.snyk.io/vuln/SNYK-JAVA-COMGOOGLECODEGSON-1730327 gson PR: https:/

[GitHub] [jclouds] gaul merged pull request #122: Removed unused jetty-server dependency

2021-10-31 Thread GitBox
gaul merged pull request #122: URL: https://github.com/apache/jclouds/pull/122 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul opened a new pull request #123: JCLOUDS-1586: Upgrade to Guice 5.0.1

2021-10-31 Thread GitBox
gaul opened a new pull request #123: URL: https://github.com/apache/jclouds/pull/123 Guice 4.2.3 makes illegal reflective accesses that Java 17 does not allow. References google/guice#1133. Release notes: https://github.com/google/guice/wiki/Guice501 -- This is an automated me

[GitHub] [jclouds] roded commented on pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-09-23 Thread GitBox
roded commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-925651690 Hi @gaul & @timuralp, I've managed to successfully replace our proprietary OAuth2 azureblob implementation with the one implemented in this PR and it seems to be working just fi

[GitHub] [jclouds-site] nacx merged pull request #19: Document Azure Blob with Azure Active Directory

2021-09-12 Thread GitBox
nacx merged pull request #19: URL: https://github.com/apache/jclouds-site/pull/19 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-un

[GitHub] [jclouds-site] nacx merged pull request #18: Remove references to SNAPSHOT builds

2021-09-12 Thread GitBox
nacx merged pull request #18: URL: https://github.com/apache/jclouds-site/pull/18 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-un

[GitHub] [jclouds-site] timuralp opened a new pull request #19: Document Azure Blob with Azure Active Directory

2021-09-12 Thread GitBox
timuralp opened a new pull request #19: URL: https://github.com/apache/jclouds-site/pull/19 Added information on the required settings to use Azure Blob with Azure Active Directory. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [jclouds-site] gaul commented on pull request #18: Remove references to SNAPSHOT builds

2021-09-11 Thread GitBox
gaul commented on pull request #18: URL: https://github.com/apache/jclouds-site/pull/18#issuecomment-917563925 Context: > Sorry, but the download page includes links to daily builds. As the page itself points out, these are not official ASF releases. > > As such, they must no

[GitHub] [jclouds-site] gaul opened a new pull request #18: Remove references to SNAPSHOT builds

2021-09-11 Thread GitBox
gaul opened a new pull request #18: URL: https://github.com/apache/jclouds-site/pull/18 jclouds is out of compliance with ASF policies by advertising SNAPSHOT builds to users. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [jclouds] gaul commented on pull request #121: JCLOUDS-1584 : Deployment API fails when ARM Template deploys

2021-09-04 Thread GitBox
gaul commented on pull request #121: URL: https://github.com/apache/jclouds/pull/121#issuecomment-912926923 Thank you for your contribution @Rajani-cloud! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [jclouds] gaul merged pull request #121: JCLOUDS-1584 : Deployment API fails when ARM Template deploys

2021-09-04 Thread GitBox
gaul merged pull request #121: URL: https://github.com/apache/jclouds/pull/121 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #120: Compute - DeployementAPI changes to fulfill ARM Template Deploy

2021-09-02 Thread GitBox
gaul commented on pull request #120: URL: https://github.com/apache/jclouds/pull/120#issuecomment-911719914 Superseded by #121. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

[GitHub] [jclouds] gaul closed pull request #120: Compute - DeployementAPI changes to fulfill ARM Template Deploy

2021-09-02 Thread GitBox
gaul closed pull request #120: URL: https://github.com/apache/jclouds/pull/120 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #121: JCLOUDS-1584 : Deployment API fails when ARM Template deploys

2021-09-02 Thread GitBox
gaul commented on pull request #121: URL: https://github.com/apache/jclouds/pull/121#issuecomment-911719662 @jmspring any chance you could look at this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [jclouds] Rajani-cloud opened a new pull request #121: JCLOUDS-1584 : Deployment API fails when ARM Template deploys

2021-09-02 Thread GitBox
Rajani-cloud opened a new pull request #121: URL: https://github.com/apache/jclouds/pull/121 JCLOUDS-1584 : Deployment API fails when ARM Template deploys for creating Virtual Machine. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [jclouds] gaul commented on pull request #120: Compute - DeployementAPI changes to fulfill ARM Template Deploy

2021-08-31 Thread GitBox
gaul commented on pull request #120: URL: https://github.com/apache/jclouds/pull/120#issuecomment-908311944 @Rajani-cloud This PR is messed up and doesn't seem to include your commit. This may be due to opening it against the 2.3.x branch. Note that we only accept PRs against the master

[GitHub] [jclouds] gaul commented on pull request #120: Compute - DeployementAPI changes to fulfill ARM Template Deploy

2021-08-30 Thread GitBox
gaul commented on pull request #120: URL: https://github.com/apache/jclouds/pull/120#issuecomment-908311944 @Rajani-cloud This PR is messed up and doesn't seem to include your commit. This may be due to opening it against the 2.3.x branch. Note that we only accept PRs against the master

[GitHub] [jclouds] Rajani-cloud opened a new pull request #120: Compute - DeployementAPI changes to fulfill ARM Template Deploy

2021-08-30 Thread GitBox
Rajani-cloud opened a new pull request #120: URL: https://github.com/apache/jclouds/pull/120 This pull request to fix the following issue https://issues.apache.org/jira/browse/JCLOUDS-1584 -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [jclouds] gaul commented on pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-08-22 Thread GitBox
gaul commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-903263720 @roded Great! I hope to release jclouds 2.4.0 next month so hopefully we can get any required changes merged. -- This is an automated message from the Apache Git Service. To respo

[GitHub] [jclouds] roded commented on pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-08-21 Thread GitBox
roded commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-903223501 I'll attempt to replace our internal implementation with this PR's and report back. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [jclouds] gaul commented on pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-08-21 Thread GitBox
gaul commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-903223131 Thanks for you contribution @timuralp! This look fine although I only tested the traditional credentials for regressions. -- This is an automated message from the Apache Git Servi

[GitHub] [jclouds] gaul merged pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-08-21 Thread GitBox
gaul merged pull request #119: URL: https://github.com/apache/jclouds/pull/119 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] roded commented on pull request #64: JCLOUDS-1520 - Reuse SSLSocketFactory in JavaUrlHttpCommandExecutorSe…

2021-08-19 Thread GitBox
roded commented on pull request #64: URL: https://github.com/apache/jclouds/pull/64#issuecomment-902470296 @gaul Due to time constraints, I don't have a path forward for this PR unfortunately. Apologies. -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [jclouds] gaul commented on pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-08-18 Thread GitBox
gaul commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-901087348 @timuralp Could you prefix the commit with JCLOUDS-1558: which will tag the issue automatically? Also please rebase the PR since I pushed the API update separately. @roded It

[GitHub] [jclouds] gaul commented on pull request #119: JCLOUDS-1558: AzureBlob: Active Directory Auth

2021-08-18 Thread GitBox
gaul commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-901087348 @timuralp Could you prefix the commit with JCLOUDS-1558: which will tag the issue automatically? Also please rebase the PR since I pushed the API update separately. @roded It

[GitHub] [jclouds] gaul commented on pull request #119: AzureBlob: Active Directory Auth

2021-08-17 Thread GitBox
gaul commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-900276216 @timuralp Please open an issue so we can capture this in the release notes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [jclouds] gaul commented on pull request #119: AzureBlob: Active Directory Auth

2021-08-17 Thread GitBox
gaul commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-900255844 @jmspring Can you help review this? @timuralp At first glance this looks fine other than the checkstyle failures. Is there some tl;dr for testing with AD? -- This is an aut

[GitHub] [jclouds] timuralp commented on pull request #119: AzureBlob: Active Directory Auth

2021-08-16 Thread GitBox
timuralp commented on pull request #119: URL: https://github.com/apache/jclouds/pull/119#issuecomment-900020002 @gaul I ran the live tests with the Azure AD credentials, but wonder if additional tests are required? Which additional unit tests would be helpful? -- This is an automated mes

[GitHub] [jclouds] timuralp opened a new pull request #119: AzureBlob: Active Directory Auth

2021-08-16 Thread GitBox
timuralp opened a new pull request #119: URL: https://github.com/apache/jclouds/pull/119 Implements the Azure AD authentication for Azure Blob, using the OAuth module. Added more parameters to the AzureBlob provider: - azureblob.auth - azureblob.account - azur

[GitHub] [jclouds] gaul commented on a change in pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
gaul commented on a change in pull request #118: URL: https://github.com/apache/jclouds/pull/118#discussion_r682461456 ## File path: blobstore/src/main/java/org/jclouds/blobstore/TransientStorageStrategy.java ## @@ -193,7 +195,14 @@ public String putBlob(final String container

[GitHub] [jclouds] timuralp commented on a change in pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
timuralp commented on a change in pull request #118: URL: https://github.com/apache/jclouds/pull/118#discussion_r683218390 ## File path: blobstore/src/main/java/org/jclouds/blobstore/TransientStorageStrategy.java ## @@ -193,7 +195,14 @@ public String putBlob(final String conta

[GitHub] [jclouds] gaul merged pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
gaul merged pull request #118: URL: https://github.com/apache/jclouds/pull/118 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
gaul commented on pull request #118: URL: https://github.com/apache/jclouds/pull/118#issuecomment-893320602 Thank you for your contribution @timuralp! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [jclouds] gaul commented on pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
gaul commented on pull request #118: URL: https://github.com/apache/jclouds/pull/118#issuecomment-893320602 Thank you for your contribution @timuralp! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [jclouds] gaul merged pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
gaul merged pull request #118: URL: https://github.com/apache/jclouds/pull/118 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] timuralp commented on a change in pull request #118: Store the MPU ETag for the transient blobstore

2021-08-05 Thread GitBox
timuralp commented on a change in pull request #118: URL: https://github.com/apache/jclouds/pull/118#discussion_r683218390 ## File path: blobstore/src/main/java/org/jclouds/blobstore/TransientStorageStrategy.java ## @@ -193,7 +195,14 @@ public String putBlob(final String conta

[GitHub] [jclouds] gaul commented on a change in pull request #118: Store the MPU ETag for the transient blobstore

2021-08-04 Thread GitBox
gaul commented on a change in pull request #118: URL: https://github.com/apache/jclouds/pull/118#discussion_r682461456 ## File path: blobstore/src/main/java/org/jclouds/blobstore/TransientStorageStrategy.java ## @@ -193,7 +195,14 @@ public String putBlob(final String container

[GitHub] [jclouds] timuralp opened a new pull request #118: Store the MPU ETag for the transient blobstore

2021-08-04 Thread GitBox
timuralp opened a new pull request #118: URL: https://github.com/apache/jclouds/pull/118 JCLOUDS-1582: fixes a bug in the transient blobstore where after uploading a multipart upload, GET/HEAD returns the hash of the content, rather than the MPU ETag. -- This is an automated message

[GitHub] [jclouds] gaul merged pull request #117: Prefer case-insenstive header comparisons

2021-07-29 Thread GitBox
gaul merged pull request #117: URL: https://github.com/apache/jclouds/pull/117 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #117: Prefer case-insenstive header comparisons

2021-07-27 Thread GitBox
gaul commented on pull request #117: URL: https://github.com/apache/jclouds/pull/117#issuecomment-887907723 @pimuzzo Please take a look. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the speci

[GitHub] [jclouds] gaul opened a new pull request #117: Prefer case-insenstive header comparisons

2021-07-27 Thread GitBox
gaul opened a new pull request #117: URL: https://github.com/apache/jclouds/pull/117 Follows up to #116. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

[GitHub] [jclouds] gaul commented on pull request #116: Fix location header case sensitive

2021-07-27 Thread GitBox
gaul commented on pull request #116: URL: https://github.com/apache/jclouds/pull/116#issuecomment-887905305 Thank you for your contribution @pimuzzo! I will fix up the other calls in a subsequent commit. -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [jclouds] gaul merged pull request #116: Fix location header case sensitive

2021-07-27 Thread GitBox
gaul merged pull request #116: URL: https://github.com/apache/jclouds/pull/116 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] pimuzzo opened a new pull request #116: Fix location header case sensitive

2021-07-26 Thread GitBox
pimuzzo opened a new pull request #116: URL: https://github.com/apache/jclouds/pull/116 Fix https://issues.apache.org/jira/browse/JCLOUDS-1569 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [jclouds] gaul merged pull request #111: Add missing javax dependencies for modern Java

2021-07-20 Thread GitBox
gaul merged pull request #111: URL: https://github.com/apache/jclouds/pull/111 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul merged pull request #114: Remove Google appengine driver

2021-07-20 Thread GitBox
gaul merged pull request #114: URL: https://github.com/apache/jclouds/pull/114 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul merged pull request #115: Remove tests which rely on an isolated ClassLoader

2021-07-20 Thread GitBox
gaul merged pull request #115: URL: https://github.com/apache/jclouds/pull/115 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #115: Remove tests which rely on an isolated ClassLoader

2021-07-15 Thread GitBox
gaul commented on pull request #115: URL: https://github.com/apache/jclouds/pull/115#issuecomment-881082042 Maybe there is a better solution to this but I'm not too familiar with `ClassLoader` issues. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [jclouds] gaul opened a new pull request #115: Remove tests which rely on an isolated ClassLoader

2021-07-15 Thread GitBox
gaul opened a new pull request #115: URL: https://github.com/apache/jclouds/pull/115 Java 9 returns `jdk.internal.loader.ClassLoaders$AppClassLoader` instead of `URLClassLoader` so we cannot create an isolated `ClassLoader`. -- This is an automated message from the Apache Git Service.

[GitHub] [jclouds] gaul merged pull request #113: Upgrade to duplicate-finder-maven-plugin 1.5.0

2021-07-15 Thread GitBox
gaul merged pull request #113: URL: https://github.com/apache/jclouds/pull/113 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #111: Add missing javax dependencies for modern Java

2021-07-14 Thread GitBox
gaul commented on pull request #111: URL: https://github.com/apache/jclouds/pull/111#issuecomment-880318121 Still iterating on this; see #113 and #114. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [jclouds] gaul opened a new pull request #114: Remove Google appengine driver

2021-07-14 Thread GitBox
gaul opened a new pull request #114: URL: https://github.com/apache/jclouds/pull/114 This has an incompatibility with JAXB motivating this removal. jclouds GAE has not seen any development or issues in recent years and uses a very old appengine-api-1.0-sdk dependency. Further it appea

[GitHub] [jclouds] gaul opened a new pull request #113: Upgrade to duplicate-finder-maven-plugin 1.5.0

2021-07-14 Thread GitBox
gaul opened a new pull request #113: URL: https://github.com/apache/jclouds/pull/113 Notably this fixes an issue with duplicate module-version files in JAXB. Release notes: https://github.com/basepom/duplicate-finder-maven-plugin/releases -- This is an automated message from th

[GitHub] [jclouds] gaul commented on pull request #89: JCLOUDS-1521: Remove need for content length when using multipart upload

2021-07-14 Thread GitBox
gaul commented on pull request #89: URL: https://github.com/apache/jclouds/pull/89#issuecomment-879909492 @radito3 Sorry for the late reply but did you try running the integration tests? When running: ``` mvn integration-test -pl :aws-s3 -Plive -Dtest.aws-s3.identity="${JCLOUDS

[GitHub] [jclouds] gaul merged pull request #112: Upgrade animal sniffer to 1.20

2021-07-14 Thread GitBox
gaul merged pull request #112: URL: https://github.com/apache/jclouds/pull/112 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul merged pull request #109: Ignore copyOf method when testing with modern Java

2021-07-14 Thread GitBox
gaul merged pull request #109: URL: https://github.com/apache/jclouds/pull/109 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul merged pull request #110: Upgrade to EasyMock 4.3

2021-07-14 Thread GitBox
gaul merged pull request #110: URL: https://github.com/apache/jclouds/pull/110 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #111: Add missing javax dependencies for modern Java

2021-07-09 Thread GitBox
gaul commented on pull request #111: URL: https://github.com/apache/jclouds/pull/111#issuecomment-877581391 Let me try upgrading animal-sniffer separately in #112. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [jclouds] gaul opened a new pull request #112: Upgrade animal sniffer to 1.20

2021-07-09 Thread GitBox
gaul opened a new pull request #112: URL: https://github.com/apache/jclouds/pull/112 This addresses a bad class file issues seen when using an explicit JAXB dependency. Release notes: https://github.com/mojohaus/animal-sniffer/releases -- This is an automated message from the A

[GitHub] [jclouds] gaul commented on pull request #109: Ignore copyOf method when testing with modern Java

2021-07-09 Thread GitBox
gaul commented on pull request #109: URL: https://github.com/apache/jclouds/pull/109#issuecomment-877546362 Curious to see if CI succeeds given the travis-ci.org turndown. We may need to do some work to move to travis-ci.com. -- This is an automated message from the Apache Git Service.

[GitHub] [jclouds] gaul commented on pull request #111: Add missing javax dependencies for modern Java

2021-07-09 Thread GitBox
gaul commented on pull request #111: URL: https://github.com/apache/jclouds/pull/111#issuecomment-877545367 This is my best guess of what we need to do but it is mostly cobbled together from StackOverflow. Let's see what CI says about older Java compatibility. -- This is an automated m

[GitHub] [jclouds] gaul opened a new pull request #111: Add missing javax dependencies for modern Java

2021-07-09 Thread GitBox
gaul opened a new pull request #111: URL: https://github.com/apache/jclouds/pull/111 Java 9 modularization removed these. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. T

[GitHub] [jclouds] gaul opened a new pull request #110: Upgrade to EasyMock 4.3

2021-07-09 Thread GitBox
gaul opened a new pull request #110: URL: https://github.com/apache/jclouds/pull/110 This upgrades asm which improves compatibility with modern Java. Release notes: https://github.com/easymock/easymock/releases -- This is an automated message from the Apache Git Service. To resp

[GitHub] [jclouds] gaul commented on pull request #64: JCLOUDS-1520 - Reuse SSLSocketFactory in JavaUrlHttpCommandExecutorSe…

2021-07-09 Thread GitBox
gaul commented on pull request #64: URL: https://github.com/apache/jclouds/pull/64#issuecomment-877542655 @roded Do we have a path forward on this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [jclouds] gaul commented on pull request #47: J clouds 1516

2021-07-09 Thread GitBox
gaul commented on pull request #47: URL: https://github.com/apache/jclouds/pull/47#issuecomment-877542616 Merging since this matches other call sites. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [jclouds] gaul commented on pull request #47: J clouds 1516

2021-07-09 Thread GitBox
gaul commented on pull request #47: URL: https://github.com/apache/jclouds/pull/47#issuecomment-877542547 Thank you for your contribution @didixith! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [jclouds] gaul merged pull request #47: J clouds 1516

2021-07-09 Thread GitBox
gaul merged pull request #47: URL: https://github.com/apache/jclouds/pull/47 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubsc

[GitHub] [jclouds] gaul opened a new pull request #109: Ignore copyOf method when testing with modern Java

2021-07-09 Thread GitBox
gaul opened a new pull request #109: URL: https://github.com/apache/jclouds/pull/109 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications

[GitHub] [jclouds] gaul merged pull request #108: JCLOUDS-1581: Make CORS maxAgeSeconds optional

2021-06-29 Thread GitBox
gaul merged pull request #108: URL: https://github.com/apache/jclouds/pull/108 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul commented on pull request #105: JCLOUDS-1577 - Allow to provide Azure Plan Information when starting …

2021-06-29 Thread GitBox
gaul commented on pull request #105: URL: https://github.com/apache/jclouds/pull/105#issuecomment-870987415 Thank you for your contribution @mnovak1! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [jclouds] gaul merged pull request #105: JCLOUDS-1577 - Allow to provide Azure Plan Information when starting …

2021-06-29 Thread GitBox
gaul merged pull request #105: URL: https://github.com/apache/jclouds/pull/105 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul opened a new pull request #108: JCLOUDS-1581: Make CORS maxAgeSeconds optional

2021-06-29 Thread GitBox
gaul opened a new pull request #108: URL: https://github.com/apache/jclouds/pull/108 This field is not required: https://cloud.google.com/storage/docs/cross-origin#cors-elements -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [jclouds] gaul merged pull request #107: Re-optimize LocalBlobStore.getBlob with ranges

2021-06-29 Thread GitBox
gaul merged pull request #107: URL: https://github.com/apache/jclouds/pull/107 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[GitHub] [jclouds] gaul opened a new pull request #107: Re-optimize LocalBlobStore.getBlob with ranges

2021-06-24 Thread GitBox
gaul opened a new pull request #107: URL: https://github.com/apache/jclouds/pull/107 This fixes a memory regression from 8de7b696e13f7131b3ea4a77b10f5cfd139dd712 where the transient `BlobStore` changed from a `ByteSource` to a `byte[]`. -- This is an automated message from the Apach

[GitHub] [jclouds] gaul commented on pull request #106: JCLOUDS-1580 - Add support for lowercase metadata headers

2021-06-22 Thread GitBox
gaul commented on pull request #106: URL: https://github.com/apache/jclouds/pull/106#issuecomment-866441222 Thank you for your contribution @eebert3! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [jclouds] gaul merged pull request #106: JCLOUDS-1580 - Add support for lowercase metadata headers

2021-06-22 Thread GitBox
gaul merged pull request #106: URL: https://github.com/apache/jclouds/pull/106 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please cont

<    1   2   3   4   5   6   7   8   9   10   >