Thank you for the prompt reply, Julien! I am really curious how well the aged setup will actually work.
Will followup with feedback
//edit: just as a short feedback:
configuring the Binding in “Sonarlint Bindings” did work
Fetching Server Side Findings probably worked (no errors in log + Last Storage Update: 26.10.22 : 16:38)
Displaying the synchronized Findings (with active Binding to SQ Server on premise) did not work
Displaying Findings from local analyze (after removal of active Binding to SQ Server) worked, but - as is expected - with the “default” ruleset of that old plugin version
So, sadly, your doubt concerning connected mode between old SonarLint Plugin and new SonarQube Server (9.6.1) was correct
Thank you for pointing me to the verbose debug logging That in itself is helpful already in dissecting the process.
I understand it as that in connected mode the analysis fails because the java-plugin from sq-server loaded and used (sonar-java-plugin-7.13.0.29990.jar - which is no more java 8 compatible) versus in standalone mode the internally supplied java-plugin is used (sonar-java-plugin-7.3.0.27589.jar - which is java 8 compatible)
But that were logentries concerning the analysis … I could not find any helpful hints why in connected mode the already serverside-stored findings would not be displayed in the older sonarlint views.
In the log i found entries that state that the server side issues were downloaded:
GET 200 https://sonarqube-instance/batch/issues?key=project.key | response time=359ms
Downloaded issues in 359ms
My current assumption is as follows:
After the issues are downloaded,
an analysis with the supplied server-config (with newer java-plugin) is triggered
which skips/removes “java-stuff” because of incompatibility with java 8
and henceforth does not display any downloaded matching java issues (because it excluded java from the results)
This is because it is not a feature of SonarLint :). SonarLint doesn’t display server-side issues. SonarLint downloads server-side issues only to perform “issue tracking” (match local and server-side issues, in order to merge some attributes, like severity, suppression, …). There is an exception with taint vulnerabilities, but I think we can ignore that for the moment.