Issue Details (XML | Word | Printable)

Key: NUCCORE-695
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Renato Garcia
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
DataNucleus Core

Support JBoss 6.0

Created: 12/Apr/11 08:42 AM   Updated: 15/Jan/12 09:25 AM   Resolved: 12/Apr/11 11:06 AM
Component/s: None
Affects Version/s: 3.0.0.m3
Fix Version/s: 3.0.0.m4

File Attachments: 1. Text File NonManagedPluginRegistry.java.patch (2 kB)

Environment: JBoss 6

Forum Thread URL: http://www.datanucleus.org/servlet/forum/viewthread_thread,6516#33663
Severity: Development


 Description  « Hide
Enhance NonManagedPluginRegistry to handle JBoss 6 new VFS version/protocol.



Renato Garcia added a comment - 12/Apr/11 08:56 AM
Patch that includes the new VFS protocol according to forum discussion and specific behaviour for JBoss 6.

I've tested under JBoss 6.0.0.Final, 5.1.0.GA, 5.0.1.GA and 4.2.3.GA with SUN JDK 1.6.0_23 and it works fine.

Renato Garcia made changes - 12/Apr/11 08:56 AM
Field Original Value New Value
Attachment NonManagedPluginRegistry.java.patch [ 11424 ]
Andy Jefferson added a comment - 12/Apr/11 11:06 AM
SVN trunk has this included. Thx!

Andy Jefferson made changes - 12/Apr/11 11:06 AM
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 3.0.0.m4 [ 11220 ]
Resolution Fixed [ 1 ]
Andy Jefferson made changes - 09/May/11 09:58 AM
Status Resolved [ 5 ] Closed [ 6 ]
Ales Justin added a comment - 15/Jan/12 12:11 AM
I think a better fix would be this -- you later don't need the InputStream details, and you also still work directly on VFS layer.
I tested it against JBossAS7 and it works (and it should also work on AS6).

        // after vfszip, vfzjar, vfsfile -- although that could probably be completely dropped in favor of this fix
        else if (pluginURL.toString().startsWith("vfs"))
        {
            String urlStr = pluginURL.toString().replace("plugin.xml", "META-INF/MANIFEST.MF");
            try
            {
                return new URL(urlStr);
            }
            catch (MalformedURLException e)
            {
                NucleusLogger.PLUGIN.warn(LOCALISER.msg("024010", urlStr), e);
                return null;
            }
        }

Andy Jefferson added a comment - 15/Jan/12 09:25 AM