Forums / Cotonti / Development / 0.9.11 pre-release testers needed

Need to test it before flagging as stable

Trustmaster
#1 2012-07-13 19:19

We have made some important changes in the users database, so we need to be sure it works and doesn't break anything before releasing 0.9.11.

So if you have time please test it on a copy of your site (to test the upgrade procedure) and also on a blank install. What to pay attention for:

  • Logging in and out of the site. Changing passwords in profile/edit, password recovery, etc.
  • Timezone selection and display.
  • Performance.

Download the latest code here. Do not apply it on live sites yet.

Thanks for testing!

May the Source be with you!
elfrenazo
#2 2012-07-14 05:19

Hello, https://weeox.com/pruebas



Logging in and out of the site ---> with https redirects me to disconnect to http

also connecting redirected to http


Changing passwords in profile/edit, password recovery, etc. ---> Error You didn't enter your present password, or it's wrong


Timezone selection and display. ---> OK



Performance ---> this site says: https://weeox.com/pruebas is not gzipped

This post was edited by elfrenazo (2012-07-14 06:15, 11 years ago)
GHengeveld
#3 2012-07-14 15:56
In case you're wondering why we ask for these specific features to be tested, there's two major changes in Siena 0.9.11:
  • New interchangable password hashing function, including a salt for greatly enhanced password security and support for more complex hashing algorithms such as blowfish or integration with SSO systems.
  • New timezone system with DST support.
elfrenazo
#4 2012-07-14 19:18

pleased to collaborate smiley

Trustmaster
#5 2012-07-23 07:51
#34962 elfrenazo:

Hello, https://weeox.com/pruebas

Logging in and out of the site ---> with https redirects me to disconnect to http

also connecting redirected to http

Changing passwords in profile/edit, password recovery, etc. ---> Error You didn't enter your present password, or it's wrong

Timezone selection and display. ---> OK

Performance ---> this site says: https://weeox.com/pruebas is not gzipped

HTTPS errors - I'll check it, thanks!

Changing passwords - I've tested and it works. Please check if you enter a correct present password.

Performance - better try this tool or this. They tell your site is gzipped.

Added 1 hour later:

HTTPS bug is now fixed. Thanks, elfrenazo!

May the Source be with you!
This post was edited by Trustmaster (2012-07-23 09:18, 11 years ago)
ez
#6 2012-07-23 16:53

Hi there,

i saw something very interesting for a backport to Genoa.... : New timezone system with DST support.

I would be very very happy if this can be arranged for Genoa too.
I allready submitted the code for improved password hashing and salting a while ago for Genoa.  (for the interested persons: http://www.cotonti.com/forums?m=posts&q=6983 )

I hope you consider this.. :)

greetings, Ez.

By the way: Maybe I have more Genoa contributions later on...

 

==- I say: Keep it EZ -==
Trustmaster
#7 2012-07-24 13:06

Hi Ez,

We have implemented password salting a bit different for Siena, see this comment. We are going to apply it on Genoa 0.6.24 too, so I hope it won't conflict with your patch.

As for timezones and DST... It isn't so easy. You see, in Siena all date/time stamps are passed throught cot_date() function which applies time format depending on current user's locale and applies timezone offset. While in Genoa  plain PHP date() function is used and offsets are calculated in place. So, backporting this feature would introduce lots of changes around all files. We try to avoid big changes in Genoa to keep all the old code working and to prevent from new bugs which appear in new code.

May the Source be with you!
ez
#8 2012-07-25 06:11

DST: hmm too bad, i really would love it if that was done for Genoa to (maybe ill try to fix that, i have an idea on how.... ill look later on how to do that)

Password: The methods you introduce are probably not compatible, but maybe i can alter my code... it all depends on how the salting is done (before or after the pass)

==- I say: Keep it EZ -==
Trustmaster
#9 2012-07-25 07:54

What do you mean by "before or after the pass"? If you mean concatenation order, then it is $pass . $salt.

May the Source be with you!
tensh
#10 2012-07-25 19:21

Maybe not entirely in topic, but I've been testing Siena 0.9.10, and:

- Adminizio theme misses main icons in the menu, which is strange.
- the reviews plugin - behaves strangely in plugin list (no title, description, I didn't install it)
- as for the checkboxlist, if I create this field and want to edit a page that previosuly didn't have this field set, the error appears (something about "is not array" - custom fields would have to check if it's empty, and if yes, then create blank array, I suppose.)
- attachments plugin should have "PFS" in requirements (when PFS is not installed, editing a page throws error)
- elRTE doesn't work on fresh install (set it in HTML parser, uninstalled CKEditor, used Chrome browser)
- there's notify for new PM, new comment (option), but not forum post?


I'm sorry it's not detailed, this is what came from my memory

This post was edited by tensh (2012-07-25 19:57, 11 years ago)
Trustmaster
#11 2012-07-26 05:48

Thanks for reporting, tensh! I'll try to fix as many of these as possible.

May the Source be with you!
tensh
#12 2012-07-26 15:29

Always at your service :) I'll try to come up with more, I'm playing with Siena right now.

Trustmaster
#13 2012-07-26 20:09
#35012 tensh:

Maybe not entirely in topic, but I've been testing Siena 0.9.10, and:

- Adminizio theme misses main icons in the menu, which is strange.
- the reviews plugin - behaves strangely in plugin list (no title, description, I didn't install it)
- as for the checkboxlist, if I create this field and want to edit a page that previosuly didn't have this field set, the error appears (something about "is not array" - custom fields would have to check if it's empty, and if yes, then create blank array, I suppose.)
- attachments plugin should have "PFS" in requirements (when PFS is not installed, editing a page throws error)
- elRTE doesn't work on fresh install (set it in HTML parser, uninstalled CKEditor, used Chrome browser)
- there's notify for new PM, new comment (option), but not forum post?


I'm sorry it's not detailed, this is what came from my memory

  • Sent a patch to Adminizio author. Waiting for him to update the package.
  • Is it a Siena or Genoa version of Reviews plugin? Did you have it installed before?
  • Sounds like a real bug, please file an issue on GitHub.
  • Fixed, added PFS dependency.
  • Fixed, quotes were wrong.
  • Right, there is no notification for forum posts yet, but GHengeveld is currently working on a new Notifier plugin so there will be.
May the Source be with you!
tensh
#14 2012-07-26 20:25

Dammit, you're right about reviews, my bad. I just filtered results by Siena plugins and blindly hit download link, not watching what's written inside reviews. Maybe the Siena reviews should be in download by default? ;)

Okay, I'll file an issue, just have to reproduce it to see exact message.

EDIT: I couldn't reproduce the error with checklistbox, probably it was fixed in the process? (I noticed it some time ago)

I installed elRTE, and the file manager. On first run file manager throws error: 

Invalid backend response.
Data is not JSON.

But Then it works normally. It throws this error when it doesn't find the datas/files folder, but just after the error, the folder is created and the file manager works normally without errors.

Additional bug:
There's a bug in elRTE: when adding images from personal file space, the editor gets duplicated, again and again every added picture. I used Chrome browser.
P.S. Also the style of image dialog looks odd - not like it should - the elFinder icon is not in place
P.S. 2 The buttons for adding color to text and background: don't work. After replacing the elrte.min.css from the original package, the color buttons started to work. So it has something to do also with that misplaced elFinder icon in image dialog.

 

This post was edited by tensh (2012-07-26 21:37, 11 years ago)
Trustmaster
#15 2012-07-27 10:32

Please start a new topic about elRTE/elFinder bugs. It is recommended to create a folder for elFinder files after configuring it in plugin settings. It is not created automatically on install because you can change it in configuration.

Added 3 days later:

Genoa 0.6.24 release candidate is ready for testing. It includes mostly the backport of hashing changes mentioned above.

May the Source be with you!
This post was edited by Trustmaster (2012-07-30 12:45, 11 years ago)