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

[Missing feature] S3 Credentials: Cannot use system IAM credentials instead of hardcoded ones #35928

Open
2 of 3 tasks
tgho-brrrr opened this issue Jun 5, 2024 · 1 comment

Comments

@tgho-brrrr
Copy link

tgho-brrrr commented Jun 5, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

Description

In the docs, it is mentioned that we can create an S3 catalog to enhance security by avoiding hardcoded credentials, and use system credentials.

However, it seems like I cannot create a catalog with type="S3" (ERROR 1105 (HY000): errCode = 2, detailMessage = Unknown catalog type: s3).

It is also not possible to create a resource without omitting the access key and the secret key:
ERROR 1105 (HY000): Unexpected exception: Access key ID cannot be blank.

Solution

IMHO, no need for an S3 catalog here, contrary to what the docs suggest.

The simplest solution would be to allow to create a resource without access key and secret key. The logic to automatically search for the credential chain is already handled it the java AWS SDK, and also seem to be handled on Doris side:

In this file: S3Util.java

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@tgho-brrrr tgho-brrrr changed the title [Missing feature] S3 catalog type not found for system credentials instead of hardcoded ones [Missing feature] S3 Credentials: catalog type s3 not found when trying to use system IAM credentials instead of hardcoded ones Jun 6, 2024
@tgho-brrrr tgho-brrrr changed the title [Missing feature] S3 Credentials: catalog type s3 not found when trying to use system IAM credentials instead of hardcoded ones [Missing feature] S3 Credentials: Cannot use system IAM credentials instead of hardcoded ones Jun 6, 2024
@tgho-brrrr
Copy link
Author

Seems to be handled in the backend as well, here.

But might need to use the default constructor here, and get rid of this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant