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.

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

Jump to bottom

Remove WebSphere Application Server support #5525

Merged
merged 13 commits into from
Nov 7, 2023
Merged

Remove WebSphere Application Server support #5525

merged 13 commits into from
Nov 7, 2023

Conversation

jkosternl
Copy link
Contributor

No description provided.

@jkosternl jkosternl self-assigned this Oct 5, 2023
@jkosternl jkosternl mentioned this pull request Oct 5, 2023
Merged
@nielsm5 nielsm5 mentioned this pull request Oct 27, 2023
Closed
34 tasks
@nielsm5 nielsm5 changed the title Feature/5524 Remove WAS support Remove WebSphere Application Server support Oct 27, 2023
@nielsm5
Copy link
Sponsor Member

nielsm5 commented Oct 27, 2023

Ik snap dat het nu een beetje onhandig is, en dat het indirect met WebSphere te maken heeft, maar het was iets fijner geweest als FilenameUtils een los pr was geweest.

@jkosternl
Copy link
Contributor Author

Ik snap dat het nu een beetje onhandig is, en dat het indirect met WebSphere te maken heeft, maar het was iets fijner geweest als FilenameUtils een los pr was geweest.

Ja, toen ik daarmee begon, wist ik ook niet dat het zo verweven was. Ik dacht dat het direct met WebSphere te maken heeft. Dat dat de enige reden is waarom men ooit een (hele) copy heeft gemaakt van een class uit een commons library.

@nielsm5
Copy link
Sponsor Member

nielsm5 commented Oct 31, 2023

Ik snap dat het nu een beetje onhandig is, en dat het indirect met WebSphere te maken heeft, maar het was iets fijner geweest als FilenameUtils een los pr was geweest.

Ja, toen ik daarmee begon, wist ik ook niet dat het zo verweven was. Ik dacht dat het direct met WebSphere te maken heeft. Dat dat de enige reden is waarom men ooit een (hele) copy heeft gemaakt van een class uit een commons library.

Dat klopt ook, maar het PR wordt zo groot 😄

@nielsm5
Remove more WAS references
6b98c72
@nielsm5
Merge branch 'master' into feature/5524_remove_WAS
f43088d 
# Conflicts:
#	aspose/src/main/java/nl/nn/adapterframework/extensions/aspose/AsposeFontManager.java
#	ear/src/main/application/META-INF/was.policy
@nielsm5
Add Security-Roles
b1e1b38
@nielsm5
Fix TestIbisTester test
cba4f54
@nielsm5 nielsm5 requested a review from tnleeuw November 5, 2023 16:00
@nielsm5 nielsm5 linked an issue Nov 6, 2023 that may be closed by this pull request
Remove WebSphere Application Server support #5524
Closed
@nielsm5 nielsm5 mentioned this pull request Nov 6, 2023
Closed
@jkosternl
Copy link
Contributor Author

@nielsm5: I've reviewed your commits and they look fine. Those were also the parts which I wasn't sure how to deal with them.

tnleeuw
tnleeuw reviewed Nov 6, 2023
View reviewed changes
ear/src/main/application/META-INF/was.policy Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Moet dit bestand niet gewoon weg?

Copy link
Contributor

Choose a reason for hiding this comment

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

Als besproken, dit bestand kan het best worden hernoemd naar frankframework.policy.

tnleeuw
tnleeuw requested changes Nov 7, 2023
View reviewed changes
Copy link
Contributor

@tnleeuw tnleeuw left a comment

Choose a reason for hiding this comment

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

Als besproken, beste om was.policy te hernoemen.

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Nov 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

70.4% 70.4% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@nielsm5 nielsm5 requested a review from tnleeuw November 7, 2023 13:11
@nielsm5 nielsm5 merged commit 84dbdbe into master Nov 7, 2023
10 of 11 checks passed
@nielsm5 nielsm5 deleted the feature/5524_remove_WAS branch November 7, 2023 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tnleeuw tnleeuw tnleeuw approved these changes

@nielsm5 nielsm5 nielsm5 approved these changes

Assignees

@jkosternl jkosternl

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove WebSphere Application Server support
3 participants
@jkosternl @nielsm5 @tnleeuw