[teiid-designer-dev] [Teiid-Designer/teiid-designer] 2478e6: TEIIDDES-2118: Factor out all connectivity plugin'...

[Paul Richardson]

  Branch: refs/heads/8.3.x
  Home:   https://github.com/Teiid-Designer/teiid-designer
  Commit: 2478e6c7f30adbd2857a53356db434799bc5abfc
      https://github.com/Teiid-Designer/teiid-designer/commit/2478e6c7f30adbd2857a53356db434799bc5abfc
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-07 (Thu, 07 Aug 2014)

  Changed paths:
    M plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/ConnectivityUtil.java
    A plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/Messages.java
    M plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/TeiidJDBCConnection.java
    A plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/messages.properties

  Log Message:
  -----------
  TEIIDDES-2118: Factor out all connectivity plugin's message strings

* Uses enums in Messages class to refactor all exception messages into
  separate resource properties file


  Commit: a93f595dfaeee07a37b21a5c851197ec27648a4d
      https://github.com/Teiid-Designer/teiid-designer/commit/a93f595dfaeee07a37b21a5c851197ec27648a4d
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-07 (Thu, 07 Aug 2014)

  Changed paths:
    M plugins/org.teiid.datatools.connectivity.ui/src/org/teiid/datatools/connectivity/ui/TeiidDriverUIContributor.java
    M plugins/org.teiid.datatools.connectivity/META-INF/MANIFEST.MF
    M plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/ConnectivityUtil.java
    M plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/TeiidJDBCConnection.java
    M plugins/org.teiid.datatools.connectivity/src/org/teiid/datatools/connectivity/security/impl/EquinoxSecureStorageProvider.java
    M plugins/org.teiid.designer.dqp.ui/src/org/teiid/designer/runtime/ui/vdb/ExecuteVdbWorker.java
    M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidConnectionInfo.java
    M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidJdbcInfo.java
    M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidServer.java
    M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidServerManager.java
    M plugins/org.teiid.designer.spi/src/org/teiid/datatools/connectivity/spi/ISecureStorageProvider.java
    M plugins/org.teiid.designer.spi/src/org/teiid/designer/runtime/spi/ITeiidConnectionInfo.java
    M tests/org.teiid.designer.dqp.test/src/org/teiid/designer/runtime/DefaultStorageProvider.java
    M tests/org.teiid.designer.dqp.test/src/org/teiid/designer/runtime/TeiidConnectionInfoTest.java

  Log Message:
  -----------
  TEIIDDES-2139 / 2237 - Adds a password token to connections and servers

* Since the url of the server is not unique, it is possible to overwrite
  passwords held in secure storage with values from other servers or
  jdbc connections.

* Adds a pass token to the servers and the connection properties which
  provides a unique reference to the password stored in secure storage

* Pass token generate with 1-way hash of the password and the url

* By carrying the pass token, each connection / server retain their unique
  password without actually holding it in the clear or revealing it until
  its actually used

* TeiidConnectionTest
 * Unit tests added for possible use cases involving password permutations
   hopefully ensuring that previous password methods of storage are taken
   into account and things stay backward compatible

* [*]SecureStorageProvider
 * Adds existence method to test a node keys existence since null is a
   valid password value

* TeiidConnectionInfo
 * Removes caching password entirely since it just confuses
 * Explicitly tests for storage password existence rather than just checking
   whether retrieved password is null
 * Only if there was a password when changing url fields is the password
   retrieved and updated with the new url


  Commit: 91fc48c8e4599d870ab16e7b27397ea81bc4975f
      https://github.com/Teiid-Designer/teiid-designer/commit/91fc48c8e4599d870ab16e7b27397ea81bc4975f
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-07 (Thu, 07 Aug 2014)

  Changed paths:
    M plugins/org.teiid.designer.dqp.ui/src/org/teiid/designer/runtime/ui/server/editor/TeiidServerEditor.java

  Log Message:
  -----------
  Miscellaneous Fixes

* TeiidServerEditor
 * Avoid executing runnable if form has already been disposed


  Commit: 35a73da05026a62837b5156e786650508ef40a19
      https://github.com/Teiid-Designer/teiid-designer/commit/35a73da05026a62837b5156e786650508ef40a19
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-07 (Thu, 07 Aug 2014)

  Changed paths:
    M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidConnectionInfo.java

  Log Message:
  -----------
  Removes unrequired debug messages in TeiidConnectionInfo


  Commit: 269fa7467b33249cbac806986bf6c14c52ccfd69
      https://github.com/Teiid-Designer/teiid-designer/commit/269fa7467b33249cbac806986bf6c14c52ccfd69
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-07 (Thu, 07 Aug 2014)

  Changed paths:
    M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidParentServerListener.java

  Log Message:
  -----------
  TEIIDES-2139 addendum

* Fixes password change failure when the admin password is modified in the
  Teiid Server Editor. Only applies to a version 7.7 server since 8+ uses
  the admin password of the parent jboss server.


Compare: https://github.com/Teiid-Designer/teiid-designer/compare/cb232b24ff64...269fa7467b33