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

"Backend Error" when setting file ACL #794

Closed
knation opened this issue Aug 12, 2015 · 8 comments
Closed

"Backend Error" when setting file ACL #794

knation opened this issue Aug 12, 2015 · 8 comments
Assignees
Labels
api: storage Issues related to the Cloud Storage API. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@knation
Copy link

knation commented Aug 12, 2015

Trying to set ACL after uploading and consistently getting a "Backend Error" when trying to set ACL. File uploads fine each time.

var image = myBucket.file('foo.jpg');
var imageWriteStream = image.createWriteStream({ contentType: 'image/jpeg' });

imageWriteStream.on('complete', function() {
  // Set ACL
  image.acl.add({
    scope: 'allUsers',
    role:  gcloud.storage.acl.READER_ROLE
  }, function(err) {
    // FAILING HERE
    console.log(err);
  });
});

my Stream.pipe(imageWriteStream);
@stephenplusplus
Copy link
Contributor

Thanks for reporting, I'll check it out!

Just confirming, are you on gcloud version 0.18?

@knation
Copy link
Author

knation commented Aug 12, 2015

Sorry about that. No, I'm on v0.15.0. The version installed through NPM. Is there an ETA of when v0.18 will be in NPM?

@stephenplusplus
Copy link
Contributor

It's actually out now. What value do you have in your package.json?

@knation
Copy link
Author

knation commented Aug 12, 2015

My bad. I must have previously installed it so it was stuck on v0.15. I updated and the problem persists.

@stephenplusplus
Copy link
Contributor

Is the messaging still Backend error? We actually changed how errors are patched from the API - maybe it's more helpful this time around. At the very least, it should be "Error during request".

@knation
Copy link
Author

knation commented Aug 12, 2015

Error: Backend Error
    at Object.parseApiResp (/Users/kirk/Projects/slashpath/src/node_modules/gcloud/lib/common/util.js:184:22)
    at Object.handleResp (/Users/kirk/Projects/slashpath/src/node_modules/gcloud/lib/common/util.js:139:25)
    at /Users/kirk/Projects/slashpath/src/node_modules/gcloud/lib/common/util.js:572:12
    at Request.onResponse [as _callback] (/Users/kirk/Projects/slashpath/src/node_modules/gcloud/node_modules/retry-request/index.js:107:7)
    at Request.self.callback (/Users/kirk/Projects/slashpath/src/node_modules/gcloud/node_modules/request/request.js:198:22)
    at Request.emit (events.js:110:17)
    at Request.<anonymous> (/Users/kirk/Projects/slashpath/src/node_modules/gcloud/node_modules/request/request.js:1057:14)
    at Request.emit (events.js:129:20)
    at IncomingMessage.<anonymous> (/Users/kirk/Projects/slashpath/src/node_modules/gcloud/node_modules/request/request.js:1003:12)
    at IncomingMessage.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)

@stephenplusplus
Copy link
Contributor

Oops, I just noticed we have incorrect docs. The parameter to use is actually entity instead of scope. Not sure that will be the fix here, but let's hope 🎌

@stephenplusplus stephenplusplus added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. docs api: storage Issues related to the Cloud Storage API. labels Aug 12, 2015
@stephenplusplus stephenplusplus self-assigned this Aug 12, 2015
@knation
Copy link
Author

knation commented Aug 12, 2015

Hey, that was it, thanks!

@knation knation closed this as completed Aug 12, 2015
sofisl pushed a commit that referenced this issue Nov 10, 2022
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [jsdoc-fresh](https://github.com/googleapis/jsdoc-fresh) | [`^1.0.1` -> `^2.0.0`](https://renovatebot.com/diffs/npm/jsdoc-fresh/1.1.1/2.0.0) | [![age](https://badges.renovateapi.com/packages/npm/jsdoc-fresh/2.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/jsdoc-fresh/2.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/jsdoc-fresh/2.0.0/compatibility-slim/1.1.1)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/jsdoc-fresh/2.0.0/confidence-slim/1.1.1)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/jsdoc-fresh</summary>

### [`v2.0.0`](https://github.com/googleapis/jsdoc-fresh/blob/HEAD/CHANGELOG.md#&#8203;200-httpsgithubcomgoogleapisjsdoc-freshcomparev111v200-2022-05-18)

[Compare Source](https://github.com/googleapis/jsdoc-fresh/compare/v1.1.1...v2.0.0)

##### ⚠ BREAKING CHANGES

-   update library to use Node 12 ([#&#8203;108](https://github.com/googleapis/jsdoc-fresh/issues/108))

##### Build System

-   update library to use Node 12 ([#&#8203;108](https://github.com/googleapis/jsdoc-fresh/issues/108)) ([e61c223](https://github.com/googleapis/jsdoc-fresh/commit/e61c2238db8900e339e5fe7fb8aea09642290182))

##### [1.1.1](https://www.github.com/googleapis/jsdoc-fresh/compare/v1.1.0...v1.1.1) (2021-08-11)

##### Bug Fixes

-   **build:** migrate to using main branch ([#&#8203;83](https://www.github.com/googleapis/jsdoc-fresh/issues/83)) ([9474adb](https://www.github.com/googleapis/jsdoc-fresh/commit/9474adbf0d559d319ff207397ba2be6b557999ac))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/nodejs-translate).
sofisl pushed a commit that referenced this issue Nov 11, 2022
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
sofisl pushed a commit that referenced this issue Jan 24, 2023
- [ ] Regenerate this pull request now.

PiperOrigin-RevId: 392067151

Source-Link: googleapis/googleapis@06345f7

Source-Link: googleapis/googleapis-gen@95882b3
sofisl pushed a commit that referenced this issue Jan 24, 2023
sofisl pushed a commit that referenced this issue Jan 25, 2023
- [ ] Regenerate this pull request now.

PiperOrigin-RevId: 392067151

Source-Link: googleapis/googleapis@06345f7

Source-Link: googleapis/googleapis-gen@95882b3
sofisl pushed a commit that referenced this issue Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the Cloud Storage API. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

2 participants