Initial implementation #1
Conversation
| {"content-type", "application/x-amz-json-1.1"} | ||
| ] | ||
|
|
||
| ExAws.Operation.JSON.new(:"cognito-idp", data: data, headers: headers) |
There was a problem hiding this comment.
@benwilson512 I tried using :cognito_idp as you suggested, but it failed with signature errors:
%{
"__type" => "InvalidSignatureException",
"message" => "Credential should be scoped to correct service: 'cognito-idp'. "
}
There was a problem hiding this comment.
I'm thinking we may want to just leave this alone as :"cognito-idp". While slightly less than ergonomic it's rare to actually need to type it, and I'm concerned that it's going to be difficult to try to normalize this at a central place.
There was a problem hiding this comment.
Yeah, it's just in 3 places, I'm completely fine with leaving it as :"cognito-idp"
mix.exs
Outdated
| @@ -0,0 +1,27 @@ | |||
| defmodule ExAws.Cognito.MixProject do | |||
There was a problem hiding this comment.
This probably should be ExAws.CognitoIdp.MixProject right?
mix.exs
Outdated
|
|
||
| def project do | ||
| [ | ||
| app: :ex_aws_cognito, |
There was a problem hiding this comment.
This should probably be :ex_aws_cognito_idp?
| {"content-type", "application/x-amz-json-1.1"} | ||
| ] | ||
|
|
||
| ExAws.Operation.JSON.new(:"cognito-idp", data: data, headers: headers) |
There was a problem hiding this comment.
I'm thinking we may want to just leave this alone as :"cognito-idp". While slightly less than ergonomic it's rare to actually need to type it, and I'm concerned that it's going to be difficult to try to normalize this at a central place.
|
@benwilson512 updated |
|
Hi @michalmuskala. If you wish to be a maintainer of |
|
Hi Ben. I was primarily interested in this since I used it at the time at a company I worked for. I moved since then and I'm not really interested in maintaining this myself. |
No description provided.