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

backup.describeBackupVault fails with permission error if backup vault does not exist. #2625

Closed
justengland opened this issue Apr 15, 2019 · 5 comments
Labels
closing-soon This issue will automatically close in 4 days unless further comments are made. guidance Question that needs advice or information. service-api This issue is due to a problem in a service API, not the SDK implementation.

Comments

@justengland
Copy link

When running a basic ___ command the command fails with an AccessDenied, it seems like this should fail with a different exception. Looks like it may be an API problem, but I am not sure.

`var backup = new AWS.Backup();

var params = {
BackupVaultName: 'fakename'
};
backup.describeBackupVault(params, function(err, data) {
if (err) console.log(err, err.stack); // If the vault does not exsist it fails with a 403 exception
else console.log(data); // if the name does exsist then it succeeds.
});`

AccessDeniedException: Insufficient privileges to perform this action.

Also the backup.describeBackupVault.promise function fails with this.makeRequest is not a function exception. I am unsure if they are related issues.

@srchase srchase added closing-soon This issue will automatically close in 4 days unless further comments are made. guidance Question that needs advice or information. labels Apr 15, 2019
@srchase
Copy link
Contributor

srchase commented Apr 15, 2019

@justengland,

I wasn't able to reproduce the promise issue.

Can you provide an example of your code?

What exception are you expecting instead of AcessDenied?

@justengland
Copy link
Author

The promise issue appears to an issue in the way was due to some async await issues on my side.

I would expect requesting a resource would yield a 404 response from the API and that would yield some sort of not found exception. Throwing a security exception, makes it seem like a security-related issue.

@no-response no-response bot removed the closing-soon This issue will automatically close in 4 days unless further comments are made. label Apr 15, 2019
@srchase srchase added service-api This issue is due to a problem in a service API, not the SDK implementation. closing-soon This issue will automatically close in 4 days unless further comments are made. labels Apr 15, 2019
@srchase
Copy link
Contributor

srchase commented Apr 15, 2019

The AccessDeniedException is returned by the Backup Service.

The Backup Service Team would be best suited to answer why a 403 is being returned instead of a 404. You can raise the issue on their AWS Developer Forum.

@no-response
Copy link

no-response bot commented Apr 22, 2019

This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.

@no-response no-response bot closed this as completed Apr 22, 2019
@lock
Copy link

lock bot commented Sep 25, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators Sep 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
closing-soon This issue will automatically close in 4 days unless further comments are made. guidance Question that needs advice or information. service-api This issue is due to a problem in a service API, not the SDK implementation.
Projects
None yet
Development

No branches or pull requests

2 participants