Skip to content

Rhino: migrate from servicemix to org.mozilla #189

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
maximthomas merged 5 commits into from
Jun 9, 2026
Merged

Rhino: migrate from servicemix to org.mozilla #189

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8485cec
Rhino: migrate from servicemix to mozilla
maximthomas Jun 9, 2026
f4291df
Add graalvm to test dependencies for openidm-repo-orientdb
maximthomas Jun 9, 2026
aa844fe
Merge branch 'issues/failing-orientdb-repo-test' into features/rhino-…
maximthomas Jun 9, 2026
b7c430c
fix failing tests
maximthomas Jun 9, 2026
74dff20
openidm-rhino bundle: update scope for Embed-Dependency
maximthomas Jun 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions openidm-repo-orientdb/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -248,6 +248,12 @@
<artifactId>mockito-all</artifactId>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.graalvm.sdk</groupId>
<artifactId>graal-sdk</artifactId>
<version>22.3.0</version>
<scope>test</scope>
</dependency>
</dependencies>
<build>
<plugins>
Expand Down
70 changes: 70 additions & 0 deletions openidm-rhino/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
* The contents of this file are subject to the terms of the Common Development and
* Distribution License (the License). You may not use this file except in compliance with the
* License.
*
* You can obtain a copy of the License at legal/CDDLv1.0.txt. See the License for the
* specific language governing permission and limitations under the License.
*
* When distributing Covered Software, include this CDDL Header Notice in each file and include
* the License file at legal/CDDLv1.0.txt. If applicable, add the following below the CDDL
* Header, with the fields enclosed by brackets [] replaced by your own identifying
* information: "Portions copyright [year] [name of copyright owner]".
*
* Copyright 2026 3A Systems LLC.
-->
<project xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<parent>
<groupId>org.openidentityplatform.openidm</groupId>
<artifactId>openidm-project</artifactId>
<version>7.0.3-SNAPSHOT</version>
</parent>

<artifactId>openidm-rhino</artifactId>
<packaging>bundle</packaging>
<name>${project.groupId}.${project.artifactId}</name>
<description>OSGi bundle wrapper for Mozilla Rhino JavaScript engine</description>
<dependencies>
<dependency>
<groupId>org.mozilla</groupId>
<artifactId>rhino</artifactId>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.mozilla</groupId>
<artifactId>rhino-engine</artifactId>
<scope>provided</scope>
</dependency>
</dependencies>
<build>
<plugins>
<plugin>
<groupId>org.apache.felix</groupId>
<artifactId>maven-bundle-plugin</artifactId>
<extensions>true</extensions>
<configuration>
<instructions>
<Bundle-Name>${project.name}</Bundle-Name>
<Bundle-SymbolicName>${project.name}</Bundle-SymbolicName>
<Embed-Dependency>*;groupId=org.mozilla;inline=true</Embed-Dependency>
<Export-Package>*</Export-Package>
Comment on lines +31 to +54

@maximthomas maximthomas Jun 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rhino components version is already declared in the commons project, see OpenIdentityPlatform/commons#181

suggestion

<Embed-Dependency>*;groupId=org.mozilla;inline=true;scope=provided</Embed-Dependency>

seems reasonable.

<Import-Package>
javax.lang.model;resolution:=optional,
javax.script;resolution:=optional,
*
</Import-Package>
<DynamicImport-Package>*</DynamicImport-Package>
<_removeheaders>
Bnd-LastModified,Built-By,Private-Package,Tool,Build-Jdk,Include-Resource,Ignore-Package,Bundle-DocURL
</_removeheaders>
</instructions>
</configuration>
</plugin>
</plugins>
</build>

</project>
5 changes: 0 additions & 5 deletions openidm-ui/openidm-ui-admin/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -187,11 +187,6 @@
<type>css</type>
</dependency>

<dependency>
<groupId>org.apache.servicemix.bundles</groupId>
<artifactId>org.apache.servicemix.bundles.rhino</artifactId>
</dependency>

<dependency>
<groupId>org.openidentityplatform.commons.ui.libs</groupId>
<artifactId>less</artifactId>
Expand Down
5 changes: 0 additions & 5 deletions openidm-ui/openidm-ui-enduser/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -139,11 +139,6 @@
<type>js</type>
</dependency>

<dependency>
<groupId>org.apache.servicemix.bundles</groupId>
<artifactId>org.apache.servicemix.bundles.rhino</artifactId>
</dependency>

<dependency>
<groupId>org.openidentityplatform.commons.ui.libs</groupId>
<artifactId>less</artifactId>
Expand Down
5 changes: 3 additions & 2 deletions openidm-util/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,9 @@
<artifactId>joda-time</artifactId>
</dependency>
<dependency>
<groupId>org.apache.servicemix.bundles</groupId>
<artifactId>org.apache.servicemix.bundles.rhino</artifactId>
<groupId>org.openidentityplatform.openidm</groupId>
<artifactId>openidm-rhino</artifactId>
<version>${project.version}</version>
</dependency>

<!-- Provided OSGi Dependencies -->
Expand Down
5 changes: 5 additions & 0 deletions openidm-zip/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,11 @@
<artifactId>openidm-script</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.openidentityplatform.openidm</groupId>
<artifactId>openidm-rhino</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.openidentityplatform.openidm</groupId>
<artifactId>openidm-api-servlet</artifactId>
Expand Down
1 change: 1 addition & 0 deletions openidm-zip/src/main/assembly/zip.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1218,6 +1218,7 @@
<exclude>jakarta.activation*</exclude>
<exclude>org.bouncycastle:bc*-fips</exclude>
<exclude>org.openjdk.nashorn:*</exclude>
<exclude>org.mozilla:rhino*</exclude>
<exclude>org.apache.felix:org.apache.felix.utils</exclude>
<exclude>org.codehaus.mojo:animal-sniffer-annotations</exclude>
</excludes>
Expand Down
7 changes: 1 addition & 6 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,6 @@
<asm.version>9.9.1</asm.version>
<!-- quartz 2.x is not compatible yet -->
<quartz.version>1.8.6_1</quartz.version>
<rhino.version>1.7.14_2</rhino.version>
<groovy.version>2.4.21</groovy.version>

<!-- OSGi/Felix versions -->
Expand Down Expand Up @@ -235,6 +234,7 @@
<!--module>openidm-workflow-remote</module-->
<module>openidm-zip</module>
<module>custom-scripted-connector-bundler</module>
<module>openidm-rhino</module>
</modules>

<profiles>
Expand Down Expand Up @@ -593,11 +593,6 @@
<artifactId>org.apache.servicemix.bundles.quartz</artifactId>
<version>${quartz.version}</version>
</dependency>
<dependency>
<groupId>org.apache.servicemix.bundles</groupId>
<artifactId>org.apache.servicemix.bundles.rhino</artifactId>
<version>${rhino.version}</version>
</dependency>
<dependency>
<groupId>org.apache.servicemix.bundles</groupId>
<artifactId>org.apache.servicemix.bundles.javax-inject</artifactId>
Expand Down
Loading