Skip to content

fix: Return error if read is called when no_rotate is false#6263

Merged
kodiakhq[bot] merged 2 commits intomainfrom
fix/return_error_no_rotate_read
Jan 3, 2023
Merged

fix: Return error if read is called when no_rotate is false#6263
kodiakhq[bot] merged 2 commits intomainfrom
fix/return_error_no_rotate_read

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

Same as #6262 not from a fork

We should return an error when trying to read if no_rotate is false as we can't read a randomly generated name

@erezrokah erezrokah requested review from a team and shimonp21 and removed request for a team January 3, 2023 10:43
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Jan 3, 2023
@erezrokah erezrokah force-pushed the fix/return_error_no_rotate_read branch from 1ea3a64 to 7a2989a Compare January 3, 2023 10:44
@erezrokah erezrokah force-pushed the fix/return_error_no_rotate_read branch from 7a2989a to e865ba9 Compare January 3, 2023 12:04
@yevgenypats yevgenypats removed the request for review from shimonp21 January 3, 2023 20:16
@vercel
Copy link
Copy Markdown

vercel bot commented Jan 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cloudquery-web 🔄 Building (Inspect) Jan 3, 2023 at 8:16PM (UTC)

@kodiakhq kodiakhq bot merged commit c475be7 into main Jan 3, 2023
@kodiakhq kodiakhq bot deleted the fix/return_error_no_rotate_read branch January 3, 2023 20:22
kodiakhq bot pushed a commit that referenced this pull request Jan 10, 2023
🤖 I have created a release *beep* *boop*
---


## [1.0.2](plugins-destination-azblob-v1.0.1...plugins-destination-azblob-v1.0.2) (2023-01-10)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/filetypes to v1.0.2 ([#6340](#6340)) ([d0d867b](d0d867b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.3 ([#6523](#6523)) ([5378f3b](5378f3b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.4 ([#6565](#6565)) ([80c7c06](80c7c06))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.18.0 ([#6339](#6339)) ([158365a](158365a))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.19.0 ([#6363](#6363)) ([ae6967c](ae6967c))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.20.0 ([#6376](#6376)) ([d6187ec](d6187ec))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.21.0 ([#6382](#6382)) ([5baea40](5baea40))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.22.0 ([#6516](#6516)) ([b7e4e73](b7e4e73))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.23.0 ([#6522](#6522)) ([ce24f1d](ce24f1d))
* **deps:** Update plugin-sdk to v1.21.0 for destinations ([#6419](#6419)) ([f3b989f](f3b989f))
* Return error if read is called when `no_rotate` is false ([#6263](#6263)) ([c475be7](c475be7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit that referenced this pull request Jan 10, 2023
🤖 I have created a release *beep* *boop*
---


## [1.0.3](plugins-destination-s3-v1.0.2...plugins-destination-s3-v1.0.3) (2023-01-10)


### Bug Fixes

* **deps:** Update module github.com/aws/aws-sdk-go-v2/config to v1.18.8 ([#6393](#6393)) ([ffba44f](ffba44f))
* **deps:** Update module github.com/aws/aws-sdk-go-v2/feature/s3/manager to v1.11.47 ([#6395](#6395)) ([71ec9b9](71ec9b9))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.2 ([#6340](#6340)) ([d0d867b](d0d867b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.3 ([#6523](#6523)) ([5378f3b](5378f3b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.4 ([#6565](#6565)) ([80c7c06](80c7c06))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.18.0 ([#6339](#6339)) ([158365a](158365a))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.19.0 ([#6363](#6363)) ([ae6967c](ae6967c))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.20.0 ([#6376](#6376)) ([d6187ec](d6187ec))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.21.0 ([#6382](#6382)) ([5baea40](5baea40))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.22.0 ([#6516](#6516)) ([b7e4e73](b7e4e73))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.23.0 ([#6522](#6522)) ([ce24f1d](ce24f1d))
* **deps:** Update plugin-sdk to v1.21.0 for destinations ([#6419](#6419)) ([f3b989f](f3b989f))
* Return error if read is called when `no_rotate` is false ([#6263](#6263)) ([c475be7](c475be7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit that referenced this pull request Jan 10, 2023
🤖 I have created a release *beep* *boop*
---


## [1.0.3](plugins-destination-gcs-v1.0.2...plugins-destination-gcs-v1.0.3) (2023-01-10)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/filetypes to v1.0.2 ([#6340](#6340)) ([d0d867b](d0d867b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.3 ([#6523](#6523)) ([5378f3b](5378f3b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.4 ([#6565](#6565)) ([80c7c06](80c7c06))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.18.0 ([#6339](#6339)) ([158365a](158365a))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.19.0 ([#6363](#6363)) ([ae6967c](ae6967c))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.20.0 ([#6376](#6376)) ([d6187ec](d6187ec))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.21.0 ([#6382](#6382)) ([5baea40](5baea40))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.22.0 ([#6516](#6516)) ([b7e4e73](b7e4e73))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.23.0 ([#6522](#6522)) ([ce24f1d](ce24f1d))
* **deps:** Update plugin-sdk to v1.21.0 for destinations ([#6419](#6419)) ([f3b989f](f3b989f))
* Return error if read is called when `no_rotate` is false ([#6263](#6263)) ([c475be7](c475be7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit that referenced this pull request Jan 10, 2023
🤖 I have created a release *beep* *boop*
---


## [1.0.3](plugins-destination-file-v1.0.2...plugins-destination-file-v1.0.3) (2023-01-10)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/filetypes to v1.0.2 ([#6340](#6340)) ([d0d867b](d0d867b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.3 ([#6523](#6523)) ([5378f3b](5378f3b))
* **deps:** Update module github.com/cloudquery/filetypes to v1.0.4 ([#6565](#6565)) ([80c7c06](80c7c06))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.18.0 ([#6339](#6339)) ([158365a](158365a))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.19.0 ([#6363](#6363)) ([ae6967c](ae6967c))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.20.0 ([#6376](#6376)) ([d6187ec](d6187ec))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.21.0 ([#6382](#6382)) ([5baea40](5baea40))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.22.0 ([#6516](#6516)) ([b7e4e73](b7e4e73))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.23.0 ([#6522](#6522)) ([ce24f1d](ce24f1d))
* **deps:** Update plugin-sdk to v1.21.0 for destinations ([#6419](#6419)) ([f3b989f](f3b989f))
* Return error if read is called when `no_rotate` is false ([#6263](#6263)) ([c475be7](c475be7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants