EXXETA / sonar-esql-plugin

Sonar plugin to analyze ESQL-sourcecode of IBM Integration Bus projects
Apache License 2.0
35 stars 34 forks source link

please make version that works with 9+ versions #145

Closed HeidiRechek closed 1 year ago

HeidiRechek commented 2 years ago

Please make this plugin work with LTS of Sonar.

ThomasPohl commented 2 years ago

The latest version is compatible with SonarQube 8.9.7 which is the current LTS version. What problems do you have running the plugin?

HeidiRechek commented 2 years ago

Ok I will attempt to run on the LTS, we just wanted to run on latest.

From: Thomas Pohl @.> Sent: Tuesday, March 15, 2022 6:15 AM To: EXXETA/sonar-esql-plugin @.> Cc: Heidi Rechek @.>; Author @.> Subject: [EXT] Re: [EXXETA/sonar-esql-plugin] please make version that works with 9+ versions (Issue #145)

The latest version is compatible with SonarQube 8.9.7 which is the current LTS version. What problems do you have running the plugin?

— Reply to this email directly, view it on GitHubhttps://github.com/EXXETA/sonar-esql-plugin/issues/145#issuecomment-1067862150, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOHYKN4APF7F4B6LO53BYTLVABWMLANCNFSM5OENBQ4Q. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.**@.>>


Disclaimer: This email and its attachments are intended only for the use of the individual or entity to which it is addressed and may contain information which is privileged, confidential, proprietary, or exempt from disclosure under applicable law. If you are not the intended recipient or the person responsible for delivering the message to the intended recipient, you are strictly prohibited from disclosing, distributing, copying, or in any way using this message. If you have received this communication in error, please notify the sender and destroy and delete any copies you may have received.

Internet email is not a 100% secure transmission medium. Please understand and observe this lack of security when emailing us. Although we have taken steps to ensure that this email and its attachments are free from any virus, it is the recipient's sole responsibility, in keeping with good computing practice, to ensure that they are actually virus free.

HeidiRechek commented 2 years ago

no ESQL quality profile shows. We want to use the branch features in version 9+ but also need esql features

HeidiRechek commented 2 years ago

we are using v9.4

tjjay17 commented 2 years ago

Are there any plans in the foreseeable future to make this compatible with versions 9+?

ThomasPohl commented 2 years ago

I'm working on a new version and will release it as soon as possible

ThomasPohl commented 2 years ago

I created a release candidate for sonarqube 9.4. Can you check if thats working for you? https://github.com/EXXETA/sonar-esql-plugin/releases/tag/3.4.0-RC1

HeidiRechek commented 2 years ago

is this specific to 9.4 or also 9.5 which was released a few weeks ago?

ThomasPohl commented 2 years ago

9.5 should also work, but I haven't tried, yet

kkalwaysok commented 2 years ago

I'm getting errors with 9.4 enterprise version.

2022-09-26T16:32:46.9825622Z INFO: Sensor ESQL Squid Sensor [iibesql] 2022-09-26T16:32:46.9826046Z WARN: ESQL sensor execute 2022-09-26T16:32:46.9867072Z INFO: 455 source files to be analyzed 2022-09-26T16:32:47.0410249Z WARN: Metric 'comment_lines_data' is deprecated. Provided value is ignored. ... 2022-09-26T16:32:48.8675656Z ##[error]ERROR: Unable to parse file: ##################/AddressValidator_StoreInDB.esql ERROR: Parse error at line 22 column 9:

kkalwaysok commented 2 years ago

@HeidiRechek did it work with 9.5?

kkalwaysok commented 2 years ago

I created a release candidate for sonarqube 9.4. Can you check if thats working for you? https://github.com/EXXETA/sonar-esql-plugin/releases/tag/3.4.0-RC1

I checked and it is not working with 9.4 enterprise.

HeidiRechek commented 2 years ago

If it works I am expecting to see it in the list of Quality Profiles . It does not show up there . I have tested in 9.5 and 9.6 (released recently)

image

HeidiRechek commented 2 years ago

I am using Developer Edition

kkalwaysok commented 2 years ago

I can see that in the list but it is throwing error message while scanning.

2022-09-26T16:32:48.8675656Z ##[error]ERROR: Unable to parse file: ##################/AddressValidator_StoreInDB.esql ERROR: Parse error at line 22 column 9:

kkalwaysok commented 2 years ago

This is the exact error.

java.lang.IllegalArgumentException: Overlapping symbol declaration and reference for symbol at Range[from [line=373, lineOffset=73] to [line=373, lineOffset=83]] at org.sonar.api.utils.Preconditions.checkArgument(Preconditions.java:43) at org.sonar.api.batch.sensor.symbol.internal.DefaultSymbolTable$DefaultSymbol.newReference(DefaultSymbolTable.java:110) at org.sonar.api.batch.sensor.symbol.internal.DefaultSymbolTable$DefaultSymbol.newReference(DefaultSymbolTable.java:104) at com.exxeta.iss.sonar.esql.highlighter.HighlightSymbolTableBuilder.addReference(HighlightSymbolTableBuilder.java:58) at com.exxeta.iss.sonar.esql.highlighter.HighlightSymbolTableBuilder.highlightSymbol(HighlightSymbolTableBuilder.java:51) at com.exxeta.iss.sonar.esql.highlighter.HighlightSymbolTableBuilder.build(HighlightSymbolTableBuilder.java:39) at com.exxeta.iss.sonar.esql.EsqlSensor.highlightSymbols(EsqlSensor.java:328) at com.exxeta.iss.sonar.esql.EsqlSensor.scanFile(EsqlSensor.java:221) at com.exxeta.iss.sonar.esql.EsqlSensor.analyse(EsqlSensor.java:157) at com.exxeta.iss.sonar.esql.EsqlSensor.analyseFiles(EsqlSensor.java:132) at com.exxeta.iss.sonar.esql.EsqlSensor.execute(EsqlSensor.java:306) at org.sonar.scanner.sensor.AbstractSensorWrapper.analyse(AbstractSensorWrapper.java:64) at org.sonar.scanner.sensor.ModuleSensorsExecutor.execute(ModuleSensorsExecutor.java:85) at org.sonar.scanner.sensor.ModuleSensorsExecutor.lambda$execute$1(ModuleSensorsExecutor.java:59) at org.sonar.scanner.sensor.ModuleSensorsExecutor.withModuleStrategy(ModuleSensorsExecutor.java:77) at org.sonar.scanner.sensor.ModuleSensorsExecutor.execute(ModuleSensorsExecutor.java:59) at org.sonar.scanner.scan.SpringModuleScanContainer.doAfterStart(SpringModuleScanContainer.java:81) at org.sonar.core.platform.SpringComponentContainer.startComponents(SpringComponentContainer.java:188) at org.sonar.core.platform.SpringComponentContainer.execute(SpringComponentContainer.java:167) at org.sonar.scanner.scan.SpringProjectScanContainer.scan(SpringProjectScanContainer.java:392) at org.sonar.scanner.scan.SpringProjectScanContainer.scanRecursively(SpringProjectScanContainer.java:388) at org.sonar.scanner.scan.SpringProjectScanContainer.doAfterStart(SpringProjectScanContainer.java:357) at org.sonar.core.platform.SpringComponentContainer.startComponents(SpringComponentContainer.java:188) at org.sonar.core.platform.SpringComponentContainer.execute(SpringComponentContainer.java:167) at org.sonar.scanner.bootstrap.SpringGlobalContainer.doAfterStart(SpringGlobalContainer.java:135) at org.sonar.core.platform.SpringComponentContainer.startComponents(SpringComponentContainer.java:188) at org.sonar.core.platform.SpringComponentContainer.execute(SpringComponentContainer.java:167) at org.sonar.batch.bootstrapper.Batch.doExecute(Batch.java:72) at org.sonar.batch.bootstrapper.Batch.execute(Batch.java:66) at org.sonarsource.scanner.api.internal.batch.BatchIsolatedLauncher.execute(BatchIsolatedLauncher.java:46) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) at java.base/java.lang.reflect.Method.invoke(Unknown Source) at org.sonarsource.scanner.api.internal.IsolatedLauncherProxy.invoke(IsolatedLauncherProxy.java:60) at com.sun.proxy.$Proxy0.execute(Unknown Source) at org.sonarsource.scanner.api.EmbeddedScanner.doExecute(EmbeddedScanner.java:189) at org.sonarsource.scanner.api.EmbeddedScanner.execute(EmbeddedScanner.java:138) at org.sonarsource.scanner.cli.Main.execute(Main.java:112) at org.sonarsource.scanner.cli.Main.execute(Main.java:75) at org.sonarsource.scanner.cli.Main.main(Main.java:61)

ThomasPohl commented 1 year ago

I finally found the problem. I had saved the declaration twice (No clue why that was no problem in previous versions) Will be fixed in next release