Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Support bootstrapping and wiping AWS Credentials #582

Merged
merged 5 commits into from
Mar 11, 2016

Conversation

RobertSzkutak
Copy link
Contributor

Issue #581

@dmcassel
Copy link
Collaborator

@RobertSzkutak I just ran a test with a non-AWS configuration and it failed.

ERROR: XDMP-AS: (err:XPTY0004) sec:credentials-set-aws($accessKey, $secretKey) -- Invalid coercion: fn:QName("","secretKey") as xs:string

The implementation of setup:create-credentials and setup:wipe-credentials needs to account for $import-config/sec:credentials not being present. If that's the case, the function should do nothing.

@RobertSzkutak
Copy link
Contributor Author

Thanks, should be fixed now!

@dmcassel
Copy link
Collaborator

Looks good. I've tested that this works fine with non-AWS configuration. I don't have an AWS setup handy, but your code looks good to me.

dmcassel added a commit that referenced this pull request Mar 11, 2016
Support bootstrapping and wiping AWS Credentials
@dmcassel dmcassel merged commit 0e98150 into marklogic-community:dev Mar 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants