-
Notifications
You must be signed in to change notification settings - Fork 0
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
AWS session token support #1
base: master
Are you sure you want to change the base?
Conversation
Sort Linux distributions and simplify/standardize grammar.
src/curl.cpp
Outdated
@@ -1194,6 +1195,17 @@ bool S3fsCurl::SetAccessKey(const char* AccessKeyId, const char* SecretAccessKey | |||
return true; | |||
} | |||
|
|||
bool S3fsCurl::SetAccessKeyWithSessionToken(const char* AccessKeyId, const char* SecretAccessKey, const char* SessionToken) | |||
{ | |||
if((!S3fsCurl::is_ibm_iam_auth && (!AccessKeyId || '\0' == AccessKeyId[0])) || !SecretAccessKey || '\0' == SecretAccessKey[0] || !SessionToken || '\0' == SessionToken[0]){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add multiple lines
bool accessKey = !AccessKeyId || '\0' == AccessKeyId[0])
...
if (!S3fsCurl::is_ibm_iam_auth && accessKey || SecretAccess
src/s3fs.cpp
Outdated
@@ -4680,6 +4710,10 @@ static int my_fuse_opt_proc(void* data, const char* arg, int key, struct fuse_ar | |||
is_ibm_iam_auth = true; | |||
return 0; | |||
} | |||
if (0 == strcmp(arg, "use_session_token")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why cannot we use STR2NCMP macro as is used bellow?
Remove custom cppcheck from ppc64le Travis config
Address shellcheck errors
Mostly small fixes but also some reordering. References s3fs-fuse#929. Found via: diff -u <(man --no-hyphenation doc/man/s3fs.1 | tr -s ' ' '\n' | sed '/^-o$/d' ) <(src/s3fs --help | tr -s ' ' '\n' | sed '/^-$/d')
560a524
to
a8e88df
Compare
Standardize installation section
Make man page and --help more consistent
AWS session token support
This prohibits missing semicolons.
This provides compatibility with C++03 and removes duplicate lookups.
Address markdownlint warnings
Prefer simple over compound statements in macros
Remove uses of std::map::at
Relevant Issue
Where to put
aws_session_token
information s3fs-fuse#651Details
AWS Session Token support