348 : if (!preg_match('/\\A\\w*\\z/', $value)) {

seems to be too strict in accepting passwords that are correct feedburner passwords and contain special characters.

Comments

shane birley’s picture

I can verify this issue since I am forced to limit the passwords I use for Feedburner since the module errors out and doesn't accept the password special characters.

dave reid’s picture

Assigned: Unassigned » dave reid
Status: Active » Postponed (maintainer needs more info)

Do either of you know what kind of special characters are allowed?

shane birley’s picture

Off the top of my head, I don't know which ones are allowed. To give you an idea, any passwords I use are generated via a tool like this: http://www.pctools.com/guides/password/

The passwords I generate from tools like this work with the Feedburner web site, but fail when I attempt to verify using the module.

Common characters like the following appear to be the issue as random passwords using only uppercase and lowercase characters work fine.

 # @ ! $ % & * _ ? + = -
shane birley’s picture

I also just did a bit of testing with password length and I don't see any problems there. But, again, only using upper and lowercase non-special characters.

joestewart’s picture

No idea about what characters they accept. But that may be part of the issue.

Stepping back a little. This also seems more strict than what Drupal itself accepts. What is the goal for this?

Looking at the code in the D6 version. I guess this has been changed/removed?

dave reid’s picture

Status: Postponed (maintainer needs more info) » Fixed

Yes I removed the code in the D6 version since right now FeedBurner is being merged to Google and the Feed Management API (which uses the account authorization) is not available. Once it becomes available again, I'll be able to code stuff that uses it. I'm going to mark this as fixed since a D5 revision will not include this code as well.

dave reid’s picture

You can keep updated on the Feed Management API with #192906: Integrate FeedBurner Management API

Anonymous’s picture

Status: Fixed » Closed (fixed)

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