Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore 405 error when S3 Acceleration is disabled on the S3 bucket #680

Merged

Conversation

sfc-gh-ext-simba-lb
Copy link
Contributor

@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb commented Oct 31, 2022

Description

Simba Salesforce ticket number 00406729.

Description provided by Snowflake Support:
When we need to upload a file to an internal stage using the PUT command it fails when GetBucketAccelerateConfiguration returns 405.

It happens when the driver tries to use the S3 acceleration service on the accounts where the S3 Acceleration is disabled on the buckets.

Observed behavior:
The driver fails with: ERRO[0000]connection.go:318 gosnowflake.(*snowflakeConn).queryContextInternal error: operation error S3: GetBucketAccelerateConfiguration, https response error StatusCode:
The file is not uploaded
In Snowflake job history the PUT query appears successful

Expected behavior:
The 405 error appears in the log
The file is uploaded
In Snowflake job history the PUT query appears successful
(This is how it is implemented in the python connector.)

Checklist

  • Code compiles correctly
  • Run make fmt to fix inconsistent formats
  • Run make lint to get lint errors and fix all of them
  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the README / documentation, if necessary

@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb changed the title ignore405Error Ignore 405 error when S3 Acceleration is disabled on the S3 bucket Oct 31, 2022
Copy link
Collaborator

@sfc-gh-igarish sfc-gh-igarish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb merged commit a2529a2 into master Nov 2, 2022
@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb deleted the ignore405ForGetBucketAccelerateConfiguration branch November 2, 2022 19:54
@github-actions github-actions bot locked and limited conversation to collaborators Nov 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants