The module works well for Google, Yahoo, OpenID, MyOpenID, AOL. Can login, and a username and email are generated.

However I am unable to get the module to work for Facebook or Twitter. In both cases the message returned is "You must enter an e-mail address.
RPX registration failed for the reasons listed. You may register now, or if you already have an account you can log in now and add your OpenID under "My Account""

1. OpenID is not enabled. I suspect this message is thrown by Drupal but the developers of the RPX module should be aware of this.
2. Since the email is not provided by Facebook (RPX Wiki says Facebook and MySpace do not provide this information in their API) then the account cannot be created and the user is presented with a dialogue to type in their email and give a password. This of course is confusing for the user because they would somehow now need to know to use the local Drupal login, which frankly I hope to hide and not have my users use because it is so much less confusing to just use RPX.

Also:
The module does not appear to create the password for the local drupal account, so anyone who uses this module must always use the RPX login. This will present confusion for the user who views their account settings and is offered a change password dialogue. I suspect this was always out of scope but worth mentioning.

Comments

nrambeck’s picture

nrambeck commented
Assigned: Unassigned » nrambeck

Significant changes were made this weekend and are available in the dev release. It retrieves the preferred username/email from the provider, but allows the user to change these before final registration. Please review the dev release and let me know if you still have issues with Facebook and Twitter.

chrisroge’s picture

chrisroge commented

This works for Facebook now however the message returned to drupal is:

"Someone has already registered that e-mail address. Please enter another, or perhaps you have already registered with a different account?"

This is not accurate as I ensured the Facebook account I used did not already have a cooresponding user account in Drupal. Can we change this message to something more like:

"A valid e-mail address is required for registration. Please provide a valid e-mail address (some open ID providers do not provide a e-mail address for privacy reasons or perhaps the e-mail address returned by [provider] is already taken for [site]."

nrambeck’s picture

nrambeck commented
Status: Active » Fixed

I added Drupal's core validation functions for username and email during the RPX registration process. If the email is missing, it will simply request that the user enter one. Changes will be available in the next nightly dev release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.