Adoption to Jira 10.0.0-m0009

atlassian-labs · Aug 7, 2024 · 9638b05 · 9638b05
1 parent ac440c8
commit 9638b05
Show file tree

Hide file tree

Showing 8 changed files with 22 additions and 15 deletions.
diff --git a/jira-slack-server-integration/jira-slack-server-integration-plugin/pom.xml b/jira-slack-server-integration/jira-slack-server-integration-plugin/pom.xml
@@ -395,6 +395,12 @@
                     <!-- Give Java more heap to make Jira faster -->
                     <jvmArgs>${jvm17.opens} -Xmx2g</jvmArgs>
 
+                    <banningExcludes>
+                        <exclude>com.atlassian.security:atlassian-secure-random</exclude>
+                        <exclude>commons-codec:commons-codec</exclude>
+                        <exclude>com.google.code.gson:gson</exclude>
+                    </banningExcludes>
+
                     <pluginArtifacts>
                         <pluginArtifact>
                             <groupId>com.atlassian.jira</groupId>
@@ -640,7 +646,7 @@
     </build>
 
     <properties>
-        <jira.amps.version>8.10.4</jira.amps.version>
+        <jira.amps.version>9.0.3</jira.amps.version>
         <project.root.directory>${project.basedir}/../..</project.root.directory>
         <maven.build.timestamp.format>yyyy-MM-dd'T'HH:mm:ss.SSSZ</maven.build.timestamp.format>
         <build.timestamp>${maven.build.timestamp}</build.timestamp>

diff --git a/...ation-plugin/src/main/java/com/atlassian/jira/plugins/slack/web/panels/IssueWebPanel.java b/...ation-plugin/src/main/java/com/atlassian/jira/plugins/slack/web/panels/IssueWebPanel.java
@@ -1,6 +1,6 @@
 package com.atlassian.jira.plugins.slack.web.panels;
 
-import com.atlassian.plugin.web.model.WebPanel;
+import com.atlassian.plugin.web.api.model.WebPanel;
 import com.atlassian.webresource.api.assembler.PageBuilderService;
 
 import java.io.IOException;

diff --git a/...-integration/jira-slack-server-integration-plugin/src/main/resources/atlassian-plugin.xml b/...-integration/jira-slack-server-integration-plugin/src/main/resources/atlassian-plugin.xml
@@ -790,4 +790,9 @@
         <url-pattern>/*</url-pattern>
         <dispatcher>REQUEST</dispatcher>
     </servlet-filter>
+
+    <velocity-allowlist key="velocity-allowlist-slack-plugin">
+        <method>com.atlassian.jira.plugins.slack.spi.impl.JiraSlackPluginResourceProvider#getPluginKey()</method>
+    </velocity-allowlist>
+
 </atlassian-plugin>
diff --git a/.../test/java/com/atlassian/jira/plugins/slack/service/task/impl/DefaultTaskBuilderTest.java b/.../test/java/com/atlassian/jira/plugins/slack/service/task/impl/DefaultTaskBuilderTest.java
@@ -24,6 +24,7 @@
 import org.mockito.junit.MockitoJUnit;
 import org.mockito.junit.MockitoRule;
 import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PowerMockIgnore;
 import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.powermock.modules.junit4.PowerMockRunner;
 
@@ -35,6 +36,7 @@
 import static org.junit.Assert.assertThat;
 
 @PrepareForTest({DefaultTaskBuilder.class, SendNotificationTask.class})
+@PowerMockIgnore({"javax.*", "org.xml.*", "org.w3c.*"})
 @RunWith(PowerMockRunner.class)
 public class DefaultTaskBuilderTest {
     @Mock

diff --git a/jira-slack-server-integration/pom.xml b/jira-slack-server-integration/pom.xml
@@ -22,12 +22,12 @@
         <aui.version>5.1.3</aui.version>
 
         <!-- this is the version we will run integration tests against -->
-        <jira.version>10.0.0-m0008</jira.version>
-        <testkit.version>8.2.2</testkit.version>
+        <jira.version>10.0.0-m0009</jira.version>
+        <testkit.version>10.0.3</testkit.version>
         <jira.test.unit.version>${jira.version}</jira.test.unit.version>
 
         <!-- This is the version the plugin is compiled against -->
-        <jira.compile.version>10.0.0-m0008</jira.compile.version>
+        <jira.compile.version>10.0.0-m0009</jira.compile.version>
 
         <jira.pageobjects.version>${jira.version}</jira.pageobjects.version>
 

diff --git a/...on-common/src/main/java/com/atlassian/plugins/slack/rest/RequestCachingServletFilter.java b/...on-common/src/main/java/com/atlassian/plugins/slack/rest/RequestCachingServletFilter.java
@@ -1,10 +1,10 @@
 package com.atlassian.plugins.slack.rest;
 
+import com.atlassian.annotations.security.UnrestrictedAccess;
 import org.apache.commons.lang3.StringUtils;
 
 import javax.servlet.Filter;
 import javax.servlet.FilterChain;
-import javax.servlet.FilterConfig;
 import javax.servlet.ServletException;
 import javax.servlet.ServletRequest;
 import javax.servlet.ServletResponse;
@@ -13,17 +13,14 @@
 
 import static com.github.seratch.jslack.app_backend.SlackSignature.HeaderNames.X_SLACK_SIGNATURE;
 
+@UnrestrictedAccess
 public class RequestCachingServletFilter implements Filter {
     private final RequestHolder requestHolder;
 
     public RequestCachingServletFilter(final RequestHolder requestHolder) {
         this.requestHolder = requestHolder;
     }
 
-    @Override
-    public void init(FilterConfig filterConfig) throws ServletException {
-    }
-
     @Override
     public void doFilter(final ServletRequest request,
                          final ServletResponse response,
@@ -40,7 +37,4 @@ public void doFilter(final ServletRequest request,
         chain.doFilter(processedRequest, response);
     }
 
-    @Override
-    public void destroy() {
-    }
 }
diff --git a/slack-server-integration-common/src/main/resources/admin/configure-slack.vm b/slack-server-integration-common/src/main/resources/admin/configure-slack.vm
@@ -1,4 +1,4 @@
-#* @vtlvariable name="webResourceManager" type="com.atlassian.plugin.webresource.api.WebResourceManager" *#
+#* @vtlvariable name="webResourceManager" type="com.atlassian.webresource.api.WebResourceManager" *#
 #* @vtlvariable name="pageBuilderService" type="com.atlassian.webresource.api.assembler.PageBuilderService" *#
 #* @vtlvariable name="webInterfaceManager" type="com.atlassian.plugin.web.WebInterfaceManager" *#
 #* @vtlvariable name="i18n" type="com.atlassian.sal.api.message.I18nResolver" *#

diff --git a/slack-server-integration-common/src/main/resources/admin/connect/connect-workspace.vm b/slack-server-integration-common/src/main/resources/admin/connect/connect-workspace.vm
@@ -1,4 +1,4 @@
-#* @vtlvariable name="webResourceManager" type="com.atlassian.plugin.webresource.api.WebResourceManager" *#
+#* @vtlvariable name="webResourceManager" type="com.atlassian.webresource.api.WebResourceManager" *#
 #* @vtlvariable name="pageBuilderService" type="com.atlassian.webresource.api.assembler.PageBuilderService" *#
 #* @vtlvariable name="webInterfaceManager" type="com.atlassian.plugin.web.WebInterfaceManager" *#
 #* @vtlvariable name="i18n" type="com.atlassian.sal.api.message.I18nResolver" *#