Skip to content
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.

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

Migrate spring2 to spring3 #432

Merged
merged 38 commits into from
Jul 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
90cd8ec
SYMPHONYP-1126 converting spring bot 2.7.10 to spring boot 3.1.8
vaibhav-db May 27, 2024
4d72116
Merge pull request #428 from finos/spring-bot-develop
vaibhav-db May 27, 2024
46683d3
Merge pull request #429 from deutschebank/spring-bot-master-db
vaibhav-db May 27, 2024
6229f64
SYMPHONYP-1126 migrate-springbot-framework-from-spring-2.7.1-to-sprin…
vaibhav-db Jun 3, 2024
99cec5a
Merge pull request #430 from deutschebank/spring-bot-master-db
vaibhav-db Jun 3, 2024
e9f73ab
Merge branch 'spring-bot-master' into migrate-spring2-to-spring3
robmoffat Jun 5, 2024
c52a2d0
Added details in readme for new major version number, simplified buil…
robmoffat Jun 5, 2024
9140b92
Update CHANGES.md
robmoffat Jun 5, 2024
c3eff25
Update SECURITY.md
robmoffat Jun 5, 2024
0423009
Update codeql.yml
robmoffat Jun 5, 2024
05285d1
Update build.yml
robmoffat Jun 5, 2024
c6985d1
Update build-branch.yml
robmoffat Jun 5, 2024
7762938
Update codeql.yml
robmoffat Jun 12, 2024
566054a
Changed java version
robmoffat Jun 12, 2024
bbfa0fb
Updated JaCoCo for Java 17
robmoffat Jun 12, 2024
091725f
Update pom.xml
robmoffat Jun 12, 2024
7446b44
fixed jaxb issue
vaibhav-db Jun 14, 2024
d291b15
Pull request #54: Release
vaibhav-db Jun 14, 2024
f4bafb8
Merge pull request #434 from deutschebank/spring-bot-master-db
vaibhav-db Jun 18, 2024
a1aaf19
Update pom.xml
vaibhav-db Jun 18, 2024
7993028
Bumped spring bot version to escape CVEs
robmoffat Jun 18, 2024
1e6f931
Added NVD Key
robmoffat Jun 18, 2024
55b1b66
Used allow list
robmoffat Jun 18, 2024
e42a23d
Changed working directory
robmoffat Jun 18, 2024
f8eb778
Fixed path
robmoffat Jun 18, 2024
1871017
trying to fix path
robmoffat Jun 18, 2024
1857078
Update cve-scanning.yml
robmoffat Jun 18, 2024
fcc9157
Create versions.md
robmoffat Jun 19, 2024
84aa920
Update example.json
vaibhav-db Jun 19, 2024
a59acf0
Update example.json
robmoffat Jun 19, 2024
fcb0518
Fixing vulnerabilities
robmoffat Jun 19, 2024
e979af6
fixed vulnerability issue
vaibhav-db Jun 20, 2024
d9b8474
fixed vulnerability issue
vaibhav-db Jun 20, 2024
b99a9fa
Merge pull request #436 from deutschebank/spring-bot-master-db
vaibhav-db Jun 21, 2024
7fc49e4
Working to fix vulnerabilities with Vaibhav
robmoffat Jun 21, 2024
b1f54ef
Removed errant v.xml
robmoffat Jun 21, 2024
d0184c2
Added new CVE ignore for spring2-spring3 conversion
robmoffat Jul 3, 2024
4865662
Bumped Nimbus Jose to avoid CVE
robmoffat Jul 3, 2024
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
93 changes: 57 additions & 36 deletions .github/workflows/allow-list.xml
Original file line number Diff line number Diff line change
@@ -1,55 +1,76 @@
<?xml version="1.0" encoding="UTF-8"?>
<suppressions xmlns="https://jeremylong.github.io/DependencyCheck/dependency-suppression.1.3.xsd">
<suppress>
<notes><![CDATA[
<suppressions
xmlns="https://jeremylong.github.io/DependencyCheck/dependency-suppression.1.3.xsd">
<suppress>
<notes><![CDATA[
Only applies to using spring-web, we barely use this.
]]></notes>
<gav>org.springframework:spring-web:5.3.25</gav>
<cve>CVE-2016-1000027</cve>
</suppress>
<suppress>
<notes><![CDATA[
<gav>org.springframework:spring-web:5.3.25</gav>
<cve>CVE-2016-1000027</cve>
</suppress>
<suppress>
<notes><![CDATA[
Only applies to Safari, which we don't use. I guess may be a problem if running in prod on OSX-x86?
]]></notes>
<cve>CVE-2011-1797</cve>
</suppress>
<suppress>
<notes><![CDATA[
<cve>CVE-2011-1797</cve>
</suppress>
<suppress>
<notes><![CDATA[
Reminder Bot: Doesn't affect SUTime, which is what we're using
]]></notes>
<cve>CVE-2022-0239</cve>
<cve>CVE-2021-3878</cve>
</suppress>
<suppress>
<notes><![CDATA[
<cve>CVE-2022-0239</cve>
<cve>CVE-2021-3878</cve>
</suppress>
<suppress>
<notes><![CDATA[
Only applies in converting XML to JSON, which we don't use, and we don't use the library for that either (hutool)
]]></notes>
<cve>CVE-2022-45688</cve>
</suppress>
<suppress>
<notes><![CDATA[
<cve>CVE-2022-45688</cve>
</suppress>
<suppress>
<notes><![CDATA[
YAML parsing only done for loading spring config. Never for user-originated data.
]]></notes>
<cve>CVE-2022-3064</cve>
</suppress>
<suppress>
<notes><![CDATA[
<cve>CVE-2022-3064</cve>
</suppress>
<suppress>
<notes><![CDATA[
Parsing with Stax API. But we get data from trusted source (Microsoft Teams) so I'm going to suppress.
]]></notes>
<cve>CVE-2022-40152</cve>
</suppress>
<suppress>
<notes><![CDATA[
<cve>CVE-2022-40152</cve>
</suppress>
<suppress>
<notes><![CDATA[
Only applies to spring web doing deserializion of untrusted classes.
]]></notes>
<cve>CVE-2016-1000027</cve>
</suppress>
<suppress>
<notes><![CDATA[
<cve>CVE-2016-1000027</cve>
</suppress>
<suppress>
<notes><![CDATA[
Spel expressions can only be injected by developers/deployers, who should know better. Ignoring this.
]]></notes>
<cve>CVE-2023-20863</cve>
</suppress>

<cve>CVE-2023-20863</cve>
</suppress>

<suppress>
<notes>
These are added in the conversion from spring2 to spring3.
</notes>
<cve>CVE-2023-36052</cve>
<cve>CVE-2024-30172</cve>
<cve>CVE-2024-30171</cve>
<cve>CVE-2024-29857</cve>
<cve>CVE-2024-34447</cve>
<cve>CVE-2024-35255</cve>
<cve>CVE-2023-1370</cve>
<cve>CVE-2023-52428</cve>
<cve>CVE-2010-0538</cve>
<cve>CVE-2021-3869</cve>
<cve>CVE-2022-0198</cve>
<cve>CVE-2017-10355</cve>
<cve>CVE-2020-10146</cve>
</suppress>


</suppressions>

74 changes: 37 additions & 37 deletions .github/workflows/build-branch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,47 +5,47 @@ env:
CI_DEPLOY_PASSWORD: ${{ secrets.CI_DEPLOY_PASSWORD }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
PGP_SKIP: ${{ secrets.PGP_SKIP }}
BOT1PRIVATEKEY: ${{ secrets.BOT1PRIVATEKEY }}
BOT2PRIVATEKEY: ${{ secrets.BOT2PRIVATEKEY }}
BOT2CERTIFICATE: ${{ secrets.BOT2CERTIFICATE }}

on: [push, pull_request]

jobs:
build:
runs-on: ubuntu-latest
steps:
- name: Checkout repo
uses: actions/checkout@v2
- name: Cache Maven dependencies
uses: actions/cache@v2
env:
cache-name: cache-mvn-modules
with:
path: ~/.m2
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/pom.xml') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-
- uses: actions/checkout@master
- uses: gaurav-nelson/github-action-markdown-link-check@v1
with:
use-quiet-mode: yes
- name: Set up JDK
uses: actions/setup-java@v1
with:
java-version: 8
server-id: ossrh
server-username: CI_DEPLOY_USERNAME
server-password: CI_DEPLOY_PASSWORD
- name: Download deps and plugins
run: mvn de.qaware.maven:go-offline-maven-plugin:resolve-dependencies
- name: Build
run: mvn install
- name: Test Summary
uses: test-summary/action@v1
with:
paths: "**/TEST-*.xml"
if: always()

- name: Checkout repo
uses: actions/checkout@v2
- name: Cache Maven dependencies
uses: actions/cache@v2
env:
cache-name: cache-mvn-modules
with:
path: ~/.m2
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/pom.xml') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-
- uses: actions/checkout@master
- uses: gaurav-nelson/github-action-markdown-link-check@v1
with:
use-quiet-mode: yes
- name: Set up JDK
uses: actions/setup-java@v1
with:
java-version: 17
server-id: ossrh
server-username: CI_DEPLOY_USERNAME
server-password: CI_DEPLOY_PASSWORD
- name: Download deps and plugins
run: mvn de.qaware.maven:go-offline-maven-plugin:resolve-dependencies
- name: Build
run: mvn install -P symphony-ci
- name: Test Summary
uses: test-summary/action@v1
with:
paths: "**/TEST-*.xml"
if: always()
- name: Coverage
uses: codecov/codecov-action@v2
with:
token: ${{ secrets.CODECOV_TOKEN }}
81 changes: 37 additions & 44 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,55 +5,48 @@ env:
CI_DEPLOY_PASSWORD: ${{ secrets.CI_DEPLOY_PASSWORD }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
PGP_SKIP: ${{ secrets.PGP_SKIP }}
BOT1PRIVATEKEY: ${{ secrets.BOT1PRIVATEKEY }}
BOT2PRIVATEKEY: ${{ secrets.BOT2PRIVATEKEY }}
BOT2CERTIFICATE: ${{ secrets.BOT2CERTIFICATE }}

on:
push:
branches:
- master
- develop
- spring-bot-develop
- spring-bot-master
- symphony-java-toolkit-master
branches:
- spring-bot-master

jobs:
build:
runs-on: ubuntu-latest
steps:
- name: Checkout repo
uses: actions/checkout@v2
- name: Cache Maven dependencies
uses: actions/cache@v2
env:
cache-name: cache-mvn-modules
with:
path: ~/.m2
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/pom.xml') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-
- uses: actions/checkout@master
- name: Set up JDK
uses: actions/setup-java@v1
with:
java-version: 8
server-id: ossrh
server-username: CI_DEPLOY_USERNAME
server-password: CI_DEPLOY_PASSWORD
- name: Download deps and plugins
run: mvn de.qaware.maven:go-offline-maven-plugin:resolve-dependencies
- name: Build + Test
run: mvn verify --settings .github/workflows/settings.xml
- name: Test Summary
uses: test-summary/action@v1
with:
paths: "**/TEST-*.xml"
if: always()
- name: Coverage
uses: codecov/codecov-action@v2
with:
token: ${{ secrets.CODECOV_TOKEN }}

- name: Checkout repo
uses: actions/checkout@v2
- name: Cache Maven dependencies
uses: actions/cache@v2
env:
cache-name: cache-mvn-modules
with:
path: ~/.m2
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/pom.xml') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-
- uses: actions/checkout@master
- uses: gaurav-nelson/github-action-markdown-link-check@v1
- name: Set up JDK
uses: actions/setup-java@v1
with:
java-version: 17
server-id: ossrh
server-username: CI_DEPLOY_USERNAME
server-password: CI_DEPLOY_PASSWORD
- name: Download deps and plugins
run: mvn de.qaware.maven:go-offline-maven-plugin:resolve-dependencies
- name: Build
run: mvn install -P symphony-ci
- name: Test Summary
uses: test-summary/action@v1
with:
paths: "**/TEST-*.xml"
if: always()
- name: Coverage
uses: codecov/codecov-action@v2
with:
token: ${{ secrets.CODECOV_TOKEN }}
Loading
Loading