[teiid-designer-dev] [Teiid-Designer/teiid-designer] bbdeaf: TEIIDDES-2139 / 2237 - Adds a password token to co...

Paul Richardson p.g.richardson at redhat.com
Wed Aug 6 06:39:05 EDT 2014


  Branch: refs/heads/8.5.x
  Home:   https://github.com/Teiid-Designer/teiid-designer
  Commit: bbdeaf4557c925ed5bc6dce502540c7401145b11
      https://github.com/Teiid-Designer/teiid-designer/commit/bbdeaf4557c925ed5bc6dce502540c7401145b11
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-06 (Wed, 06 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: e00652d3f58c58b6281ff4cc787a148f09ec6128
      https://github.com/Teiid-Designer/teiid-designer/commit/e00652d3f58c58b6281ff4cc787a148f09ec6128
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-06 (Wed, 06 Aug 2014)

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

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

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

* ExecutionAdmin
 * pingJDBC should not call admin.deploy but adminSpec.deploy since the
   deploy method changed names between Teiid 7.7 and 8.0


  Commit: d032ea1bea10b5d998c21d2cf0a87bac12edc459
      https://github.com/Teiid-Designer/teiid-designer/commit/d032ea1bea10b5d998c21d2cf0a87bac12edc459
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-06 (Wed, 06 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: 33bde2db12a95acc020d92bd84b5cade00b8117c
      https://github.com/Teiid-Designer/teiid-designer/commit/33bde2db12a95acc020d92bd84b5cade00b8117c
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2014-08-06 (Wed, 06 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/1023f1cf7bc1...33bde2db12a9


More information about the teiid-designer-dev mailing list