Taverna 2

CredentialManager does not support duplicate priorities for MasterPasswordProviderSPI

Details

Description

The credential manager sorts MasterPasswordProviderSPI implementations by putting them in a TreeMap by the priority returned from canProvidePassword() - however this means that any providers with the same priority (which they might not know off) would overwrite eachother in the map.

Activity

Hide
Stian Soiland-Reyes added a comment - 2010-01-29 14:28

done for 2.1.1 - sort as a List instead

Show
Stian Soiland-Reyes added a comment - 2010-01-29 14:28 done for 2.1.1 - sort as a List instead
Hide
Hudson Daemon added a comment - 2010-01-29 14:32

Integrated in net.sf.taverna.t2.ui-components #1196
T2-698 return negative priority if on headless

Use priority 100 instead of 1 to allow some more room between this provider and 0.

Show
Hudson Daemon added a comment - 2010-01-29 14:32 Integrated in net.sf.taverna.t2.ui-components #1196 T2-698 return negative priority if on headless Use priority 100 instead of 1 to allow some more room between this provider and 0.
Hide
Hudson Daemon added a comment - 2010-01-29 14:32

Integrated in net.sf.taverna.t2.security #417
T2-698 don't show master password dialogue if headless - instead check system property

T2-1094 Also tests the default certchain password 'changeit' (for OSX 10.6), "" and the system property javax.net.ssl.trustStorePassword

support duplicate MasterPasswordProviderSPIs - sort List using Collections.sort and a comparator instead.

Show
Hudson Daemon added a comment - 2010-01-29 14:32 Integrated in net.sf.taverna.t2.security #417 T2-698 don't show master password dialogue if headless - instead check system property T2-1094 Also tests the default certchain password 'changeit' (for OSX 10.6), "" and the system property javax.net.ssl.trustStorePassword support duplicate MasterPasswordProviderSPIs - sort List using Collections.sort and a comparator instead.
Hide
Hudson Daemon added a comment - 2010-02-04 16:08

Integrated in net.sf.taverna.t2.security #423
T2-698 T2-T2-1094 T2-1124 T2-1127 T2-1129 T2-1133 T2-1137 T2-1143 T2-1145 Updated version numbers to 1.0.1 instead of 1.0.1-SNAPSHOT
T2-698 T2-T2-1094 T2-1124 T2-1127 T2-1129 T2-1133 T2-1137 T2-1143 T2-1145 patches for 2.1.1

Show
Hudson Daemon added a comment - 2010-02-04 16:08 Integrated in net.sf.taverna.t2.security #423 T2-698 T2-T2-1094 T2-1124 T2-1127 T2-1129 T2-1133 T2-1137 T2-1143 T2-1145 Updated version numbers to 1.0.1 instead of 1.0.1-SNAPSHOT T2-698 T2-T2-1094 T2-1124 T2-1127 T2-1129 T2-1133 T2-1137 T2-1143 T2-1145 patches for 2.1.1
Hide
Alan Williams added a comment - 2010-03-05 15:55

Tagging was messed up in the patch

Show
Alan Williams added a comment - 2010-03-05 15:55 Tagging was messed up in the patch
Hide
Alan Williams added a comment - 2010-03-30 12:09

Confirmed it is done as list

Show
Alan Williams added a comment - 2010-03-30 12:09 Confirmed it is done as list

People

Vote (0)
Watch (0)

Dates

  • Created:
    2010-01-29 11:25
    Updated:
    2010-03-30 12:09
    Resolved:
    2010-03-09 11:44