Posted by EvanDonovan on March 23, 2012 at 8:58pm
13 followers
Jump to:
| Project: | AES encryption |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | reviewed & tested by the community |
Issue Summary
I just fixed some of the most crucial bugs with AES for Drupal 7. Before releasing again, can some people test and confirm that the -dev branch is working in all the common use cases? I want 7.x-1.6 to be both stable and secure.
Comments
#1
#2
Thx. Will do. Might be nice to post dev links directly on the aes module page (i.e. without having to go to "view all releases").
#3
Seems to work fine. I did not test it extensively, but at least the admin form and file-based storage works as expected.
#4
Seems to work fine for our site too. Although we did not completely test all functions. But day-to-day functionality gave no errors.
#5
A dev release isn't available from the releases section on the project homepage, would be great to have one there for easy access.
#6
Doxygen and comment formatting conventions
Under this aspect, the code is complete buggy.
Use single quotes instead of double quote (Drupals quotes).
You can use the Coder module to check many things above.
#7
Please fix the aes.install file.
It exist
#8
I'm currently testing the latest 7.x code as pulled from the repo on a site that uses AES with the Commerce UPS (http://www.drupal.org/project/commerce_ups) module. So far, no issues.
@quiptime - the issue you're seeing has been fixed in the lastest code from the repo (although there's no -dev release yet...)
-mike
#9
Sounds like there has been sufficient testing, then? I'll try to release a new build of the module in the next few days, then.
#10
(Based on the comment from #8 saying bug #2 in there is fixed.)
I'm aware of the Coder issues, having inherited this module from someone else; sorry I haven't been maintaining more since the original patches were committed.
#11
I just made a -dev release which should be coming from the 7.x-1.x branch. Let me know if there are any issues with that.
#12
Will 7.x-1.6 be released soon?
#13
@svouthi: It might take a while, because I am supposed to release the version after doing some testing and ensuring its stability. but right now I am not finding enough time. It might take a month or two or the release to come out.
#14
Thanks for your reply, bobbyaldol. I am appreciative of the module already, to be sure. The August "almost there" posts got my hopes up, so I thought to ask.