violations-lib icon indicating copy to clipboard operation
violations-lib copied to clipboard

Use OWASP Java Encoder

Open jeremylong opened this issue 2 years ago • 2 comments

The OWASP Java Encoder is a high performance encoding library. While I understand this is adding a dependency - the encoder project itself has not dependencies and is a very stable library.

jeremylong avatar May 20 '22 19:05 jeremylong

If the tripple brackets solves this, the PR can be closed: https://github.com/tomasbjerre/violation-comments-lib/pull/22#issuecomment-1133535735

Or else, I think this library needs to be relocated in the build.gradle:

project.ext.buildConfig = [
  publishing: [
    relocate: [
      'com.google',
      'com.jakewharton',
      'org.owasp'
    ]
  ],

tomasbjerre avatar May 21 '22 05:05 tomasbjerre

Thank you for pointing out the triple bracket syntax. Yes, that solves my current issue. However, this PR was not specifically to fix the newline issue I was having. While yes, it does introduce a new library - the library is faster, compliant with the RFC, and does not have any transitive dependencies. But I also understand the desire to use fewer libraries. Totally fine if you don't want to introduce a dependency.

jeremylong avatar May 21 '22 09:05 jeremylong