Closed (fixed)
Project:
FeedBurner
Version:
5.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
16 Jun 2008 at 20:07 UTC
Updated:
16 Sep 2008 at 05:52 UTC
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
Comment #1
shane birley commentedI 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.
Comment #2
dave reidDo either of you know what kind of special characters are allowed?
Comment #3
shane birley commentedOff 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.
Comment #4
shane birley commentedI 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.
Comment #5
joestewart commentedNo 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?
Comment #6
dave reidYes 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.
Comment #7
dave reidYou can keep updated on the Feed Management API with #192906: Integrate FeedBurner Management API
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.