You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Logging: although the code is commented out, so it's not relevant (org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase#iLogger)
Skipping the Expression Language (EL) related processing in org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase#propertiesHelper: again this is only relevant if there are keys with ".elConfig" suffix
The following lines of code:
//InputStream inputStream = configFile.getConfigFileType().inputStream(configFile.getConfigFileTypeConfig(), this);try {
//get the string and store it first (to see if it changes later)StringconfigFileContents = configFile.retrieveContents(this);
configFile.setContents(configFileContents);
result.properties.load(newStringReader(configFileContents));
in org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase#retrieveFromConfigFiles which seem to do the same as the original code.
The question is, are these modifications really needed? If not, the original code could be used as a maven dependency:
Side note: the Grouper project is outdated/bulky/poorly written with a lot of duplicated code from the org.apache.commons:commons-lang3 and other common libraries. It would be nice to replace with a better alternative
The text was updated successfully, but these errors were encountered:
The following classes:
org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase
(original source code)org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeCommonUtils
org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeUtils
were copied from the Grouper repository.
It seems that only a few changes has been made:
Logging: although the code is commented out, so it's not relevant (
org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase#iLogger
)Skipping the Expression Language (EL) related processing in
org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase#propertiesHelper
: again this is only relevant if there are keys with ".elConfig" suffixThe following lines of code:
in
org.owasp.csrfguard.config.overlay.ConfigPropertiesCascadeBase#retrieveFromConfigFiles
which seem to do the same as the original code.The question is, are these modifications really needed? If not, the original code could be used as a maven dependency:
Side note: the Grouper project is outdated/bulky/poorly written with a lot of duplicated code from the
org.apache.commons:commons-lang3
and other common libraries. It would be nice to replace with a better alternativeThe text was updated successfully, but these errors were encountered: