[Libreoffice-qa] [REVIEWED] Backport reading AES-encrypted ODF 1.2 documents (as genereated by LibO 3.5)

2011-11-17 Thread Michael Meeks
Hi Stephan,

On Fri, 2011-11-11 at 10:50 +0100, Stephan Bergmann wrote:
 http://pkgs.fedoraproject.org/gitweb/?p=libreoffice.git;a=blob;f=Backport-reading-AES-encrypted-ODF-1.2-documents.patch;h=e6c722598ab05464f09787355b621cbb0aa07c49;hb=e7a803540d408adab3d55fb2ae051ac4be599a72
  
 is a patch (actually, four separate patches for the components, 
 lib-core, lib-gui, and ure repos) to backport support for reading (but 
 not writing) AES-encrypted ODF 1.2 documents to libreoffice-3-4.

Lovely ! :-) I finally got to review it.

 If people are happy with the linked patch: fine.  If people would prefer 
 a cherry-picking approach, I could post a list of relevant commits 

So - I think we don't have the resource to do much more than this.
Clearly we are going to want some to ask for some QA focus in 3.4.next
around document encryption [ particularly interacting with signed
documents from 3.5 ].

 (technically, I produced the patch in a different way, more or less 
 applying mav60 to libreoffice-3-4 directly, so do not have that list 
 handy).  (And if there are objections against including this in the 3.4 
 code line at all, that would of course be fine as well.)

Sorry for the delay; of course it is a big diff - in particular the
rampant header movement is particularly unhelpful for being able to
actually read what changed ;-) The removal of the XMemoryStream stuff is
interesting too, I assume that is made redundant by some other seekable
caching / fallback means. The property name defines clean up is nice but
not needed.

Anyhow - assuming it has been tested thoroughly (and I assume so since
it's mostly 3.5 code being back-ported), and assuming we get some heavy
QA for the next 3.4.x release in this area - I think we should get it
in. The user experience regression of not being able to open newly
encrypted documents - and with such an hideous error message (that we
get without this fix) is just too bad :-)

Out of interest - has Fedora shipped this yet ? and what was the
feedback like ?

Thanks !

Michael.

-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
List Name: Libreoffice-qa mailing list
Mail address: Libreoffice-qa@lists.freedesktop.org
Change settings: http://lists.freedesktop.org/mailman/listinfo/libreoffice-qa
Problems? http://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/
Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette
List archive: http://lists.freedesktop.org/archives/libreoffice-qa/


Re: [Libreoffice-qa] [REVIEWED] Backport reading AES-encrypted ODF 1.2 documents (as genereated by LibO 3.5)

2011-11-17 Thread Caolán McNamara
On Thu, 2011-11-17 at 10:10 +, Michael Meeks wrote:
   Out of interest - has Fedora shipped this yet ? and what was the
 feedback like ?

It's in rawhide, i.e. will-be-F17, so its out there in our development
series with no obvious cockups yet, but no massive testing yet in
production series.

C.

___
List Name: Libreoffice-qa mailing list
Mail address: Libreoffice-qa@lists.freedesktop.org
Change settings: http://lists.freedesktop.org/mailman/listinfo/libreoffice-qa
Problems? http://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/
Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette
List archive: http://lists.freedesktop.org/archives/libreoffice-qa/