Re: CheckStyle Rule Change

2021-12-15 Thread Ivan Kelly
Ignore this, looked further and it actually does double resolution. sheesh On Wed, Dec 15, 2021 at 10:14 AM Ivan Kelly wrote: > > Maybe this has been discussed elsewhere, but if possible, we should > add a rule that to block any future log4shell type attack area. > In human terms, the rule should

Re: CheckStyle Rule Change

2021-12-15 Thread Ivan Kelly
Maybe this has been discussed elsewhere, but if possible, we should add a rule that to block any future log4shell type attack area. In human terms, the rule should be: Disallow any {log,logger,LOG}.* with a non-const first argument. -Ivan On Wed, Dec 15, 2021 at 7:15 AM Enrico Olivelli wrote: >

Re: CheckStyle Rule Change

2021-12-14 Thread Enrico Olivelli
Il Mer 15 Dic 2021, 06:38 ZhangJian He ha scritto: > Hello everyone, I have added checkstyle on pulsar-metadata, and > UnusedImports check on test code. If your PR is involved in these two > situations. Please pull the master code. Thanks > Got it Thanks Enrico > > Thanks > ZhangJian He >

CheckStyle Rule Change

2021-12-14 Thread ZhangJian He
Hello everyone, I have added checkstyle on pulsar-metadata, and UnusedImports check on test code. If your PR is involved in these two situations. Please pull the master code. Thanks Thanks ZhangJian He