payton

Results 70 comments of payton

I will look into this. Personally, I'm not very familiar with Cognito. Pull requests are always welcome, too. 😄

AWS Cognito should be added soon. As far as I can tell, this branch should be working as is https://github.com/Widen/cloudfront-auth/tree/cognito I would like to test a bit more before merging...

Agreed. This may involve a bit of a larger change. The original concept was that we decouple authentication and authorization such that the user is first authenticated and then authorized....

Great! If you'd like to make a PR, I'd be happy to review and merge it :)

Ultimately, you would want to add another Lambda function that executes on Viewer Response. This function would then modify/add the relevant headers. I'll leave this issue open as a feature...

Ah... looks like we're dependent on #41 Let's keep this issue open as a next step once the referenced issue is resolved.

@dougglez Let us know if you have any issues after running the tests

@dougglez Just following up on this issue. Any updates on your end?

I've never received this error before. First, is there a reason why your redirect URI is a file? We suggest it be `/_callback`

Sorry if I confused you with this. I mean your redirect URI should be "http://{cloudfront-endpoint}.net/_callback". Then you must add that same redirect URI to the list of accepted redirect URIs...