You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Sebb (JIRA)" <ji...@apache.org> on 2016/06/29 21:29:10 UTC

[jira] [Resolved] (CRYPTO-92) Handling default properties; allow SystemProperties to be ignored

     [ https://issues.apache.org/jira/browse/CRYPTO-92?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sebb resolved CRYPTO-92.
------------------------
       Resolution: Fixed
    Fix Version/s: 1.0.0

CRYPTO-92 Handling default properties; allow SystemProperties to be
ignored

Only use the provided property instance.
Fixes #64

Project: http://git-wip-us.apache.org/repos/asf/commons-crypto/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-crypto/commit/9182668c
Tree: http://git-wip-us.apache.org/repos/asf/commons-crypto/tree/9182668c
Diff: http://git-wip-us.apache.org/repos/asf/commons-crypto/diff/9182668c


> Handling default properties; allow SystemProperties to be ignored
> -----------------------------------------------------------------
>
>                 Key: CRYPTO-92
>                 URL: https://issues.apache.org/jira/browse/CRYPTO-92
>             Project: Commons Crypto
>          Issue Type: Improvement
>            Reporter: Sebb
>             Fix For: 1.0.0
>
>
> The code currently checks both local Properties and SystemProperties in several places.
> This is unnecessary, as the Properties class already supports the notion of property defaults, as these can be passed in to the constructor.
> I think it would be better to drop the specific check of the SystemProperties.
> This would be more flexible, as the user could decide whether or not to allow the use of SystemProperties or some other set of defaults.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)