Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

Migrate from Log4j to Reload4j #986

Closed
GCHQDeveloper314 opened this issue Mar 25, 2022 · 1 comment · Fixed by #992
Closed

Migrate from Log4j to Reload4j #986

GCHQDeveloper314 opened this issue Mar 25, 2022 · 1 comment · Fixed by #992

Comments

@GCHQDeveloper314
Copy link
Member

All versions of Gaffer are moving to Reload4j. Some dependencies in Gaffer Tools modules use log4j, which will need to be excluded to prevent a conflict.
This only requires minor POM changes to exclude log4j from some dependencies which use it, changes will be a smaller version of this.

@GCHQDeveloper314 GCHQDeveloper314 added this to the v1.22.0 milestone Mar 25, 2022
@GCHQDeveloper314 GCHQDeveloper314 self-assigned this Mar 25, 2022
t92549 added a commit that referenced this issue Mar 25, 2022
* gh-984: Change usage of log4j to slf4j

* gh-984: Checkstyle

* Exclude Log4j dependencies

Co-authored-by: t92549 <[email protected]>
@t92549 t92549 linked a pull request Mar 25, 2022 that will close this issue
@t92549
Copy link
Contributor

t92549 commented Mar 25, 2022

Closed by #992

@t92549 t92549 closed this as completed Mar 25, 2022
t92549 added a commit that referenced this issue Mar 25, 2022
* Updated Gaffer version to 1.22.0

* Gh-986: Migrate log4j to reload4j (#992)

* gh-984: Change usage of log4j to slf4j

* gh-984: Checkstyle

* Exclude Log4j dependencies

Co-authored-by: t92549 <[email protected]>

* gh-975: Fix gafferpy alpha 1 (#994)

Co-authored-by: Gaffer <[email protected]>
Co-authored-by: GCHQDeveloper314 <[email protected]>
t92549 added a commit that referenced this issue May 12, 2023
* Revert "gh-951: Add deprecation note to readme (#952)" (#983)

This reverts commit 94a4d38.

* gh-984: Change usage of log4j to slf4j (#985)

* gh-984: Change usage of log4j to slf4j

* gh-984: Checkstyle

* Updated Gaffer version to 1.22.0 (#990)

* Updated Gaffer version to 1.22.0

* Gh-986: Migrate log4j to reload4j (#992)

* gh-984: Change usage of log4j to slf4j

* gh-984: Checkstyle

* Exclude Log4j dependencies

Co-authored-by: t92549 <[email protected]>

* gh-975: Fix gafferpy alpha 1 (#994)

Co-authored-by: Gaffer <[email protected]>
Co-authored-by: GCHQDeveloper314 <[email protected]>

* prepare release gaffer-tools-1.22.0

* prepare for next development iteration

* Update links to point to v1docs (#999)

* Graph layout is now saved when navigate away from page (#812)

* Graph layout is now saved when navigate away from page

* Fixed issue with old graph results still showing after clear

* Updating removed elements to use concat

* Updating to use forEach

Updating removePreviouslyRemovedElements to use forEach

* Updated syntax error

* Altered foreach syntax due to jasmine errors

* gh-496: Fix bug with concat

Co-authored-by: t11947 <[email protected]>
Co-authored-by: t92549 <[email protected]>

* Add Deprecation message to UI README (#1061)

* Updated Gaffer version to 1.23.0 (#1064)

* Updated Gaffer version to 1.23.0

* CommonConstants replaced

* Fixed js deps

* Fixed js deps again

* Original package.json

---------

Co-authored-by: Gaffer <[email protected]>

* prepare release gaffer-tools-1.23.0

* prepare for next development iteration

---------

Co-authored-by: Gaffer <[email protected]>
Co-authored-by: GCHQDeveloper314 <[email protected]>
Co-authored-by: p013570 <[email protected]>
Co-authored-by: t11947 <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants