Just fixed.. now the separator is included again..

Bye,
Norman


2011/9/2 Eric Charles <[email protected]>:
> Hi Sai,
>
> It seems that your fix made the encryption work. Thx again!
>
> However, mail bodies are not more displayed by mail clients, although the
> source of the mail contains the body.
>
> The mail format seems strange: no separation before the body, a ) ending the
> body.
>
> This is even the case if no encryption is used.
>
> I have to double-check all this, but I'm running out-of-time today, and
> tomorrow will also be a busy day. I hope to come back to it this weekend.
>
> Thx.
>
>
> On 01/09/11 08:39, Saibabu Vallurupalli wrote:
>>
>> What I have got it - mailbox-store 0.3 and mailbox-jpa from trunk do you
>> think there is some inconsistency between these two modules? The reason
>> behind is Norman integrated Encryption code to trunck so I have to build
>> JPA
>> from Trunk and use it. Rest of the modules all are part of James 3.0 beta
>> 2
>> release.
>>
>> Thanks, Sai
>>
>> On Thu, Sep 1, 2011 at 11:19 AM, Eric
>> Charles<[email protected]>wrote:
>>
>>> ouch, not sure if move is broken or not...
>>> Need more time to review this.
>>> Eric
>>>
>>>
>>> On 01/09/11 08:13, Eric Charles wrote:
>>>
>>>> Running with encryption gives issue while moving mails (at least getting
>>>> the content of a folder after mails has been moved to it).
>>>>
>>>> I tried the same with standard (no encryption) and it gives the same
>>>> issue.
>>>>
>>>> There are no excpetion in log.
>>>>
>>>> The good news is that it's unrelated to encryption.
>>>> The bad news is that mail move seems broken in trunk.
>>>>
>>>> Sai, Can you reproduce this also (without encryption)?
>>>>
>>>> Thx.
>>>>
>>>> On 01/09/11 07:45, Eric Charles wrote:
>>>>
>>>>> Hi Sai,
>>>>> I took a little time to further look.
>>>>> No need to change code to return JPAEncryptedMessage.
>>>>> OpenJPAMailboxManager does it depending on the constructor called by
>>>>> spring (with a boolean or with a string).
>>>>>
>>>>> I have now changed
>>>>> https://svn.apache.org/repos/**asf/james/mailbox/trunk/**
>>>>> spring/src/main/resources/**META-INF/org/apache/james/**
>>>>>
>>>>> spring-mailbox-jpa.xml<https://svn.apache.org/repos/asf/james/mailbox/trunk/spring/src/main/resources/META-INF/org/apache/james/spring-mailbox-jpa.xml>
>>>>>
>>>>> to test and I'll report back the result.
>>>>>
>>>>> Thx.
>>>>>
>>>>> On 01/09/11 03:50, Eric Charles wrote:
>>>>>
>>>>>> Hi Sai,
>>>>>>
>>>>>> When you run the tests in the mailbox-jpa module, there are no
>>>>>> persistence.xml, but all the JPA properties are set via code.
>>>>>>
>>>>>> If you look at the 3 classes in
>>>>>> http://svn.apache.org/repos/**asf/james/mailbox/trunk/jpa/**
>>>>>>
>>>>>> src/test/java/org/apache/**james/mailbox/jpa/<http://svn.apache.org/repos/asf/james/mailbox/trunk/jpa/src/test/java/org/apache/james/mailbox/jpa/>
>>>>>>
>>>>>>
>>>>>> , you will see for example
>>>>>>
>>>>>> properties.put("openjpa.**MetaDataFactory", "jpa(Types=" +
>>>>>> JPAMailbox.class.getName() + ";" +
>>>>>> AbstractJPAMessage.class.**getName() + ";" +
>>>>>> JPAMessage.class.getName() + ";" +
>>>>>> JPAProperty.class.getName() + ";" +
>>>>>> JPAUserFlag.class.getName() + ";" +
>>>>>> JPASubscription.class.getName(**) + ")");
>>>>>>
>>>>>> This is where you have to change to add the JPAEncryptedMessage.
>>>>>>
>>>>>>
>>>>>>
>>>>>> When you run the server as a whole, you need to build the app module.
>>>>>>
>>>>>> In this case, the file to change is
>>>>>> http://svn.apache.org/repos/**asf/james/app/trunk/src/main/**
>>>>>>
>>>>>> config/james/META-INF/**persistence.xml<http://svn.apache.org/repos/asf/james/app/trunk/src/main/config/james/META-INF/persistence.xml>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> You will do 'mvn package', and the generated zip/tar.gz will contain
>>>>>> that file.
>>>>>>
>>>>>>
>>>>>> I hope you can make it, even if I am a bit late for you work night.
>>>>>>
>>>>>> Thx.
>>>>>>
>>>>>>
>>>>>> On 31/08/11 18:53, Saibabu Vallurupalli wrote:
>>>>>>
>>>>>>> Hi Eric,
>>>>>>>
>>>>>>> O.k.. I agree with you. But I need little bit more details regarding
>>>>>>> what
>>>>>>> should I do for the path. Since, I have the environment I can try
>>>>>>> this
>>>>>>> locally and also deploy in our Test environment and perform quick
>>>>>>> regression
>>>>>>> testing.
>>>>>>>
>>>>>>> Path:
>>>>>>> JPAMainboxManager.java is using JPAMessage object I need to modify
>>>>>>> the
>>>>>>> code
>>>>>>> to use JPAEncryptedMessage and return the same.
>>>>>>>
>>>>>>> I am trying this locally and the tests failed, Now my understanding
>>>>>>> is I
>>>>>>> also need to change persistence.xml file. Since, I am not 100% used
>>>>>>> to
>>>>>>> the
>>>>>>> development environment I need to know how Maven uses this file and
>>>>>>> from
>>>>>>> where this file will be picked?
>>>>>>>
>>>>>>> Can you please give me some insight about the path? I will be working
>>>>>>> tonight to get this patch out. If you could help me I really
>>>>>>> appreciate you.
>>>>>>> I am in USA EST timezone.
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Sai
>>>>>>>
>>>>>>>
>>>>>>> On Wed, Aug 31, 2011 at 7:27 PM, Eric
>>>>>>> Charles<eric.charles@u-**mangate.com<[email protected]>
>>>>>>>>
>>>>>>>> wrote:
>>>>>>>
>>>>>>>  Hi Sai,
>>>>>>>>
>>>>>>>> If you change JPAMessage to JPAEncryptedMessage in the
>>>>>>>> JPAMessageManager,
>>>>>>>> you must update persistence.xml accordingly.
>>>>>>>>
>>>>>>>> However, even if OpenJPAMailboxManager handles the encryption
>>>>>>>> feature to
>>>>>>>> initialize the EncryptionDecryptHelper if needed, the
>>>>>>>> JPAMailboxManager
>>>>>>>> should also verify the feature, and return a JPAEncryptedMessage if
>>>>>>>> needed.
>>>>>>>>
>>>>>>>> I hope this makes sense, and if yes, a patch is needed.
>>>>>>>>
>>>>>>>> wdyt?
>>>>>>>>
>>>>>>>>
>>>>>>>> On 31/08/11 15:19, Saibabu Vallurupalli wrote:
>>>>>>>>
>>>>>>>>  Hi Eric/Norman:
>>>>>>>>>
>>>>>>>>> I am able to reproduce this error in a simple way. In my
>>>>>>>>> checked-out
>>>>>>>>> version
>>>>>>>>> -
>>>>>>>>> C:\work\workspace\james\****mailbox\trunk\jpa\src\main\**
>>>>>>>>> java\org\apache\james\mailbox\****jpa\JPAMessageManager.java
>>>>>>>>> There is reference to JPAMessage.java class and I changed it to
>>>>>>>>> JPAEncryptedMessage.java ran the maven clean package and the tests
>>>>>>>>> failed
>>>>>>>>> with the similar exception:
>>>>>>>>>
>>>>>>>>> Thought of letting you. Thank you very much in advance.
>>>>>>>>>
>>>>>>>>> Thanks, Sai
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>  ______________________________****_______________
>>>>>>>>>>
>>>>>>>>>> *From:* Valluripalli, Sai
>>>>>>>>>> *Sent:* Wednesday, August 31, 2011 2:42 PM
>>>>>>>>>> *To:* '[email protected]'
>>>>>>>>>> *Subject:* Unable to move emails between folders with Encryption
>>>>>>>>>> turned -
>>>>>>>>>> on
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Hi Norman/Eric,
>>>>>>>>>>
>>>>>>>>>> We were in our System Testing phase and close to our release.
>>>>>>>>>> During our
>>>>>>>>>> System Testing we discovered that Move option is not working
>>>>>>>>>> between the
>>>>>>>>>> folders. When we try to move email from Inbox to Trash we
>>>>>>>>>> encounter
>>>>>>>>>> the
>>>>>>>>>> exception below. I have also got the complete log if you need. But
>>>>>>>>>> the
>>>>>>>>>> root
>>>>>>>>>> cause I understood from the below exception is somewhere in the
>>>>>>>>>> mail/store
>>>>>>>>>> code and other places JPAMessage class is referenced and when I
>>>>>>>>>> changed
>>>>>>>>>> it
>>>>>>>>>> to JPAEncryptedMessage it is unable to locate JPAMessage and
>>>>>>>>>> throwing the
>>>>>>>>>> exception.
>>>>>>>>>>
>>>>>>>>>> Is there any master configuration to modify or do I need to refer
>>>>>>>>>> JPAEncryptedMessage.java class to any other modules and re-compile
>>>>>>>>>> it?
>>>>>>>>>> Please advise.
>>>>>>>>>>
>>>>>>>>>> Caused by:<openjpa-2.1.0-r422266:****1071316 fatal user error>
>>>>>>>>>> org.apache.openjpa.****persistence.ArgumentException: No metadata
>>>>>>>>>> was
>>>>>>>>>> found for
>>>>>>>>>> type "class org.apache.james.mailbox.jpa.****
>>>>>>>>>> mail.model.openjpa.JPAMessage"****. The
>>>>>>>>>> class does not appear in the list of persistent types:
>>>>>>>>>> [org.apache.james.mailbox.jpa.****mail.model.JPAUserFlag,
>>>>>>>>>> org.apache.james.mailbox.jpa.****mail.model.JPAMailbox,
>>>>>>>>>> org.apache.james.mailbox.jpa.****mail.model.openjpa.****
>>>>>>>>>> JPAEncryptedMessage,
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> org.apache.james.domainlist.****jpa.model.JPADomain,
>>>>>>>>>> org.apache.james.mailbox.jpa.****mail.model.openjpa.****
>>>>>>>>>> AbstractJPAMessage,
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> org.apache.james.user.jpa.****model.JPAUser,
>>>>>>>>>> org.apache.james.mailbox.jpa.****mail.model.JPAProperty,
>>>>>>>>>> org.apache.james.rrt.jpa.****model.JPARecipientRewrite,
>>>>>>>>>> org.apache.james.mailbox.jpa.****user.model.JPASubscription].
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.meta.****MetaDataRepository.****
>>>>>>>>>> getMetaDataInternal(**
>>>>>>>>>>
>>>>>>>>>> MetaDataRepository.java:407)
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.meta.****MetaDataRepository.****getMetaData(**
>>>>>>>>>> MetaDataRepository.java:384)
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.kernel.****BrokerImpl.persistInternal(**
>>>>>>>>>> BrokerImpl.java:2601)
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.kernel.****BrokerImpl.persist(BrokerImpl.**
>>>>>>>>>> **java:2540)
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.kernel.****BrokerImpl.persist(BrokerImpl.**
>>>>>>>>>> **java:2437)
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.kernel.****DelegatingBroker.persist(**
>>>>>>>>>> DelegatingBroker.java:1069)
>>>>>>>>>> at
>>>>>>>>>> org.apache.openjpa.****persistence.EntityManagerImpl.****
>>>>>>>>>> persist(EntityManagerImpl.****java:707)
>>>>>>>>>> at sun.reflect.****NativeMethodAccessorImpl.****invoke0(Native
>>>>>>>>>> Method)
>>>>>>>>>> at
>>>>>>>>>> sun.reflect.****NativeMethodAccessorImpl.****invoke(**
>>>>>>>>>> NativeMethodAccessorImpl.java:****39)
>>>>>>>>>> at
>>>>>>>>>> sun.reflect.****DelegatingMethodAccessorImpl.****invoke(**
>>>>>>>>>> DelegatingMethodAccessorImpl.****java:25)
>>>>>>>>>> at java.lang.reflect.Method.****invoke(Method.java:597)
>>>>>>>>>> at
>>>>>>>>>> org.springframework.orm.jpa.****ExtendedEntityManagerCreator$****
>>>>>>>>>> ExtendedEntityManagerInvocatio****nHandler.invoke(**
>>>>>>>>>> ExtendedEntityManagerCreator.****java:365)
>>>>>>>>>> at $Proxy56.persist(Unknown Source)
>>>>>>>>>> at
>>>>>>>>>> org.apache.james.mailbox.jpa.****mail.JPAMessageMapper.save(**
>>>>>>>>>> JPAMessageMapper.java:362)
>>>>>>>>>> ... 100 more
>>>>>>>>>> DEBUG 12:55:29,285 | james.imapserver | ID=1684154803 Error while
>>>>>>>>>> processing imap request
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> We appreciate your help on this.
>>>>>>>>>>
>>>>>>>>>> Thank you, Sai.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>  --
>>>>>>>>
>>>>>>>> Eric
>>>>>>>> http://about.echarles.net
>>>>>>>>
>>>>>>>> --
>>>>>>>> Eric
>>>>>>>> http://about.echarles.net
>>>>>>>>
>>>>>>>> ------------------------------****----------------------------**
>>>>>>>> --**---------
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> To unsubscribe, e-mail:
>>>>>>>> server-dev-unsubscribe@james.****apache.org<server-dev-**
>>>>>>>>
>>>>>>>> [email protected]<[email protected]>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> For additional commands, e-mail:
>>>>>>>> [email protected].****org<server-dev-help@james.**
>>>>>>>> apache.org<[email protected]>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>> --
>>> Eric
>>> http://about.echarles.net
>>>
>>
>
> --
> Eric
> http://about.echarles.net
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to