Add bg_response in request payload to passwd challenge

[ezeqme]

This fix resolves the issue reported on #248

When adding the --bg-response parameter, the value is not passed on in the captcha validation request, which causes a validation error.

The error occurs because google blocks the request in new installations due to the default value that informs that there is no support for javascript "bg_response" with value "js_disabled".

It is necessary to pass the value as "bg_response" with value "js_enabled" to avoid the error in the google authentication flow.

But the error continues to occur in the captcha validation because this parameter was not passed on in the challenge response payload.

By adding "bg_response" with value "js_enabled" in the challenge submission, the authentication flow proceeds normally.

This error occurs in new installations of aws-google-auth, not being reproduced in installations that already existed, I can't explain why.

[viniciusgbatista]

It works

[pichouk]

Fixes the issue for me too. Thanks a lot @ezequielsbarros
Now the question is : who have write access to this repository and to PyPI ? Because the last merge / release is from 15 months ago 😬

[gurumaia]

Fixes our problem as well. @stevemac007 can you get this approved?

[jindriago-scf]

Hi team, is there something we can do to help merge this PR? This solves the problem some of our engineers are having deploying to AWS.

[ezequielbarros]

Hi team, is there something we can do to help merge this PR? This solves the problem some of our engineers are having deploying to AWS.

Temporarily while this PR is not merged I have locally altered the google.py file including the two lines of code from the commit of this PR. You can direct your engineers to do the same to temporarily fix the problem.

[luizliso]

Now it wokrs! Thanks @ezequielbarros

[jeanmercierswile]

It has worked for me. 👍

[akours]

Works like a charm! 👍

[rdonkin-attest]

We are getting this issue a lot - this PR worked for several people to fix.

Hope this can be merged soon!

[mrmooon]

In the meantime, while this PR gets merged, we've been able to workaround this using the Docker image, which is still working (at least for most people in our team).

[LeandroMReis]

Worked perfectly. Thanks @ezequielbarros

[LeandroMReis]

@stevemac007

[igorGevaerd]

It worked like a charm!

[yh0an]

It works thanks ❤️

[jonathanmdr]

It works! Thanks @ezequielsbarros

[KlavsKlavsen]

Soo missing a maintainer here :(

[dienp]

Please merge

[pichouk]

Hi everyone :)
As you can all see this project is not maintained anymore. So I'm pretty sure that adding new approvals, "+1", or whatever will not result in anything else than sending annoying notification to everyone 😉

If someone have the time/energy to maintain a fork, or to reach a maintainer (or someone at Cevo Australia) in order to get write access to this project and to PyPi, it could be great. But I suggest to discuss this on a dedicated Github issue or on #238. Because I don't think this PR will be merged and release with the current situation.

[danmoz]

I see we got it merged, and even a new release on PyPI -- thanks to all involved!

One last thing, does anyone have access to push an updated image to Dockerhub?

I've built and pushed up my own version at danmorrison/aws-google-auth... but I'd sure prefer to be using the "official" one.