Good point David. That is certainly one of the issues with the patch, in that it lumps a number of changes together. If it were broken up i would be fine with saying parts of it could go in and wait on only smaller parts of it.

On Fri, Jun 15, 2012 at 8:10 AM, David Winslow <dwinslow@opengeo.org> wrote:
Is it feasible to reduce the size of the patch by including only the consistency fixes, and shipping a GeoServer 2.2 release that knowingly contains UI bugs and uses deprecated (but presumed working) code?

--
David Winslow
OpenGeo - http://opengeo.org/

On Fri, Jun 15, 2012 at 9:50 AM, Christian Mueller <mcrmcr21@gmail.com> wrote:
Hi Andrea,

Yes, imho GSIP 77 will improve the situation. Big +1 for the proposal. The whole new security system is a monster change  and I do not want to have a Geoserver release with an inconsistent security system.  Let us wait for Justins opinion, I would vote for your proposal having a beta 3.

Christian


2012/6/15 Andrea Aime <andrea.aime@geo-solutions.it>
On Fri, Jun 15, 2012 at 1:05 PM, Christian Mueller <mcrmcr21@gmail.com> wrote:
> The question is how to continue, two facts I want to point out
>
> - We cannot make a 2.2.0 release without the changes. The system would not
> work correctly.
> - My next steps would be to review/complete the security documentation and
> during this work, make a next round hardening the code.
>
>
> Opinions ?.

The fact that we still need a 4400+loc patch to fix the security
subsystem tells me
whatever we release next week cannot possibly be a release candidate, but at
best a beta3, especially since you say that a next round of hardening is in
the plans: nothing bad about it per se, but bad that it's needed since RC means
Release Candidate, means we believe we're done and ask the users to
check and eventually tell us otherwise.

Hopefully GSIP 77 will bring some sanity into all of this.

Btw, I have no time to review the patch, I can have a look during the weekend
but my familiarity with the new authentication system is not enough.
I'll trust Justin's judgement on it unless my quick review really
finds some red flag.

Cheers
Andrea

--
Ing. Andrea Aime
GeoSolutions S.A.S.
Tech lead

Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy

phone: +39 0584 962313
fax:      +39 0584 962313
mob:    +39 339 8844549

http://www.geo-solutions.it
http://geo-solutions.blogspot.com/
http://www.youtube.com/user/GeoSolutionsIT
http://www.linkedin.com/in/andreaaime
http://twitter.com/geowolf


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel



------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel




--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.