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

Fix aws ruby and minio-js behavior when invalid endpoint is passed #172

Merged

Conversation

kannappanr
Copy link
Collaborator

Make sure that aws-sdk-ruby and minio-js report failure,when
wrong credentials are passed.

Fixes #164

Make sure that aws-sdk-ruby and minio-js report failure,when
wrong credentials are passed.

Fixes minio#164
@nitisht nitisht added this to the Current milestone Oct 2, 2017
Copy link
Contributor

@nitisht nitisht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, LGTM

Status: Downloaded newer image for play.minio.io/mint:travis-dc309243a1af5f7813e8bd6badabcf2324e02305
To get intermittent logs, 'sudo docker cp b62040b146d1:/mint/log /tmp/mint-logs'
Running aws-sdk-php tests ... FAILED in 0 seconds
Running aws-sdk-ruby tests ... FAILED in 1 seconds
Running awscli tests ... FAILED in 1 seconds
Running mc tests ... FAILED in 1 seconds
Running minio-go tests ... FAILED in 0 seconds
Running minio-java tests ... FAILED in 0 seconds
Running minio-js tests ... FAILED in 1 seconds
Running minio-py tests ... FAILED in 0 seconds
Finished running all tests.
To get logs, run 'sudo docker cp b62040b146d1:/mint/log /tmp/mint-logs'

Copy link
Member

@harshavardhana harshavardhana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@nitisht nitisht merged commit fa5fc9b into minio:master Oct 2, 2017
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

Successfully merging this pull request may close these issues.

None yet

3 participants