Hi , I whould also like to backport: JAMES-757: MailAddress not check for valid syntax if new MailAddress(user, domain) is used
bye Norman Norman Maurer schrieb: > Stefano Bagnara schrieb: > >> I use -0.9 instead of -1 because I don't want them to be considered >> vetoes even if I don't think it is a good idea to backport them. >> >> Norman Maurer wrote: >> >>> Hi Guys, >>> >>> i finished the copy and merge of the 2.3.1 branch. I updated the headers >>> too. Now there are some issues I whould like to backport to 2.3.1.. So >>> please review and give me some input ;-) >>> >>> JAMES-715: RemoteDelivery act invalid domains as temporary failure >>> >> -0.9 DNSServer has changed a lot since 2.3.0 and a backport of this >> bugfix will need a big rewrite of the patch. I think this and >> JAMES-672 will have to wait for next-minor or next-major. >> > > Well i think its a "critical bug". At least i think so.. So i whould > prefer to merge it if its possible .. But maybe you are right and this > is not possible with current DNSServer, because we depend on the Lookup > result .. > > >>> JAMES-721: MBoxMailRepository.remove(String key) causes >>> ClassCastException >>> >> +1 >> >> >>> JAMES-672: DNSServer does not properly cache MX lookups for CNAMEd >>> domains >>> >> This has been fixed in a big "rewrite" of the DSNServer to fix a bunch >> of bugs and properly use the Lookup class removing every static access >> to the Lookup object. >> IIRC this change needs an assembly.xml change and cannot be backported >> to 2.3.1. Maybe this can be done in next-minor. >> > > Ok assembly.xml change is not acceptable :-/ > > >>> Optional: >>> JAMES-743: Add "with ESMTP" to received header if EHLO was used >>> >> -0 it is a feature and not a blocking bug and we didn't test it too >> much to understand wether this can give problems in the real world or >> not. >> >> >>> JAMES-753: Support ESMTPA , ESMTPS , ESMTPSA headers >>> >> -0 same as above >> >> >>> JAMES-652: Add info in about HELO/EHLO in "Received:" Header >>> >> I don't remember if this is a bugfix or this is a feature. My rule of >> thumb is: if it is a bugfix then ok to include, otherwise wait for >> next-minor/next-major. >> >> >>> JAMES-577: Switch default sendpartial to true for RemoteDelivery >>> >> -0.9 this can be changed in xml, and i think it is not good to change >> a default value like this in a point release. It might change expected >> behaviour. >> >> >>> I only listed JIRA issues which are not allready assigned to 2.3.1. >>> > > Thats why i marked them optional.. I don't care to much of this issues. > > > >> I would add the following: >> >> JAMES-671: ServerConnection doesn't properly handle the TCP/IP backlog >> https://issues.apache.org/jira/browse/JAMES-671 >> >> +1 >> > > +1 > > >> JAMES-730: [patch] add postgres support t sqlResources file for >> Bayesian analysis >> https://issues.apache.org/jira/browse/JAMES-730 >> >> +0.5 >> > > +1 Makes sense to me and its a low risk change. > >> JAMES-515: Remove RepositoryManager and cornerstone-store-impl >> https://issues.apache.org/jira/browse/JAMES-515 >> (This was vetoed by Vincenzo while we was in 2.3RC: I list it anyway >> because maybe the approach is different now). >> >> +0.5 >> > > + 0.2 Not so importent.. Its just a cleanup so i don't care to much of > this in 2.3.1 > > >> We should also consider moving this bugs to 2.3.1: >> >> JAMES-663 - UNRESOLVED - sendmail.py crashes on line "from_addr = >> os.environ['USER'] + '@' + socket.getfqdn()" >> > > +1 > > >> JAMES-662 - UNRESOLVED - sendmail.py doesn't handle multiple >> TO-recipients >> > > +1 > > >> JAMES-679 - UNRESOLVED - Make sure our container use an expiration for >> cached dns data >> >> > > +1 > > >> Stefano >> > > > Thx for adding these issues.. I forgot them :-( > > bye > Norman > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
