Skip to content

Feature : default server loaded from preferences#6

Closed
axel3rd wants to merge 2 commits into
SonarSource:masterfrom
axel3rd:feature-default-servers
Closed

Feature : default server loaded from preferences#6
axel3rd wants to merge 2 commits into
SonarSource:masterfrom
axel3rd:feature-default-servers

Conversation

@axel3rd
Copy link
Copy Markdown

@axel3rd axel3rd commented Aug 13, 2013

Hi,

Proposition for this little feature to be able to load default SonarQube servers by -pluginCustomization eclipse process, using the proxy credentials if exists (in an organisation with centralised authentification system, the proxy and SonarQube authentification could be the same).

Notes :

  • ScopedPreferenceStore can't iterate on a node but only get keys, so "new default system key" is required with index incrementation.
  • Alone thread reference : stackoverflow

TODO (by me if idea accepted) :

  • Check why apache commons lang StringUtils seems very long to load at first access
  • Complete org.sonar.ide.eclipse.core.tests for this feature (currently I develop with Eclipse Indigo and tycho integration gives me some compilation errors ... I need investigate with Kepler)

If you have suggestions or other strategy to realize this feature, I can change the orientation of my code.

@simonbrandhof
Copy link
Copy Markdown
Contributor

Thanks for the contribution. We will take a look on this pull request during the next sprint of Eclipse plugin.

@henryju
Copy link
Copy Markdown
Member

henryju commented May 20, 2014

I have created following JIRA ticket to not forget: http://jira.codehaus.org/browse/SONARIDE-428

@henryju henryju self-assigned this Jun 10, 2015
@henryju
Copy link
Copy Markdown
Member

henryju commented Jun 22, 2015

I have implemented this a bit differently (using regular Eclipse mechanism for default values). I didn't took the part about getting credentials from the proxy settings since in my opinion it is too heavily coupled with your organization internals). Anyway thanks for the PR.

@henryju henryju closed this Jun 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants