marsevilspirit opened a new pull request, #2944: URL: https://github.com/apache/dubbo-go/pull/2944
## Description SonarCloud is reporting a false-positive security warning on the use of `rand.Int63n` within the `TestConcurrentStreams` function. Using a non-cryptographically secure random number generator is acceptable and intended in this test context. The previous attempt to suppress the warning using a combined comment (`//nolint: gosec //NOSONAR`) was not effective for SonarCloud. It appears that the Sonar scanner fails to parse its `//NOSONAR` directive when it's placed after another linter's suppression comment on the same line. This PR resolves the issue by removing the `//nolint: gosec` part, leaving only the `//NOSONAR` directive. This allows SonarCloud to correctly recognize the suppression instruction and helps to clean up the CI analysis report. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org For additional commands, e-mail: notifications-h...@dubbo.apache.org
