Skip to content

Replace Apache Commons Codec Base64 usage with JDK Base64#378

Merged
kevin-mcgoldrick merged 7 commits intomasterfrom
RemoveApacheCommonsCodec
May 1, 2025
Merged

Replace Apache Commons Codec Base64 usage with JDK Base64#378
kevin-mcgoldrick merged 7 commits intomasterfrom
RemoveApacheCommonsCodec

Conversation

@kevin-mcgoldrick
Copy link
Collaborator

Replace Apache Commons Codec Base64 usage with JDK Base64

introduced in Java 8

Base64 Encoding Performance: JDK vs Apache Commons

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

* master:
  Fix Authentication in LoginFilter (#381)
  Bump org.apache.tomcat:tomcat-catalina from 11.0.2 to 11.0.5 (#382)
  Auto-populate Tank Token in Agent Debugger Tool  (#380)
  Restore JSF Routing to the Login Flow (#379)

# Conflicts:
#	web/web_support/src/main/java/com/intuit/tank/util/RestSecurityFilter.java
* master:
  Fix ICE (#393)
  version updates (#392)
  Fix content-type on jobs/script/{jobid} (#389)
  InsufficientInstanceCapacity Fix (#391)
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro left a comment

Choose a reason for hiding this comment

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

LGTM

@kevin-mcgoldrick kevin-mcgoldrick merged commit 5433405 into master May 1, 2025
3 checks passed
@kevin-mcgoldrick kevin-mcgoldrick deleted the RemoveApacheCommonsCodec branch May 1, 2025 17:36
kevin-mcgoldrick added a commit that referenced this pull request May 1, 2025
* master:
  Replace Apache Commons Codec Base64 usage with JDK Base64 (#378)
  banner in primfaces:messages (#383)
  Return 202 when applicable (#390)
  Fix BouncyCastle Proxy Usage (#394)
  Reduce Login Logging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants