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.

Sign up for GitHub

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

Jump to bottom

S3 provider always uses us-east-1 #8

Closed
ooswald opened this issue Dec 16, 2020 · 2 comments
Closed

S3 provider always uses us-east-1 #8

ooswald opened this issue Dec 16, 2020 · 2 comments

Comments

@ooswald
Copy link

ooswald commented Dec 16, 2020 •
edited

I am not sure wether this change was correct:
c6cddae#diff-ec36749d97d730e19702baa9729041e766dd4134df981ae67702657c0abd515b

Because in 0.4.2 I get this when fetching s3 buckets:

2020/12/17 00:04:54 AuthorizationHeaderMalformed: The authorization header is malformed; 
the region 'us-east-1' is wrong; expecting 'eu-central-1'
@ghost ghost closed this as completed in a15afdf Dec 17, 2020
@yevgenypats yevgenypats reopened this Dec 17, 2020
@ghost
Copy link

ghost commented Dec 17, 2020

@ooswald thanks for bringing this up. I think it is fixed now in v0.4.3 , I wasn't able to reproduce the exact same error message, can you please try and re-run this to see if the error is solved?

@ooswald
Copy link
Author

ooswald commented Dec 17, 2020

Awesome, I am happy to confirm this S3 issue is resolved - also the other commit which resolved the access key panic.

@ooswald ooswald closed this as completed Dec 17, 2020
erezrokah pushed a commit that referenced this issue Aug 14, 2022
@cq-bot
chore(main): Release v0.0.3 (#8)
595c5ff 
🤖 I have created a release *beep* *boop*
---


## [0.0.3](https://github.com/cloudquery/cq-provider-fuzz/compare/v0.0.2...v0.0.3) (2022-06-26)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.12.2 ([#7](https://github.com/cloudquery/cq-provider-fuzz/issues/7)) ([f9d3e92](https://github.com/cloudquery/cq-provider-fuzz/commit/f9d3e92df35131c0ca8229c60f1d7387d30210e9))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
erezrokah pushed a commit that referenced this issue Aug 14, 2022
@roneli
initial commit update github provider (#8)
5a7ac1f 
* initial commit update github provider

* goimports

* add tables

* add team repos

* add issues

* add billing tables

* add organizations installations

* add organizations installations

* adding hook resource

* add mock services

* start adding mock tests

* fix imports

* more tests

* external groups mock tests

* finish up tests

* update table docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ooswald @yevgenypats