Skip to content

Commit e5f5184

Browse files
authored
Merge pull request #1139 from dstd/patch-1
Fix of incorrect configuration change validation
2 parents 561b5ef + 833df18 commit e5f5184

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

app/java/net/openid/appauthdemo/Configuration.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,9 @@ private void readConfiguration() throws InvalidConfigurationException {
192192
Buffer configData = new Buffer();
193193
try {
194194
configSource.readAll(configData);
195+
final String configHash = configData.sha256().base64();
195196
mConfigJson = new JSONObject(configData.readString(Charset.forName("UTF-8")));
197+
mConfigHash = configHash;
196198
} catch (IOException ex) {
197199
throw new InvalidConfigurationException(
198200
"Failed to read configuration: " + ex.getMessage());
@@ -201,7 +203,6 @@ private void readConfiguration() throws InvalidConfigurationException {
201203
"Unable to parse configuration: " + ex.getMessage());
202204
}
203205

204-
mConfigHash = configData.sha256().base64();
205206
mClientId = getConfigString("client_id");
206207
mScope = getRequiredConfigString("authorization_scope");
207208
mRedirectUri = getRequiredConfigUri("redirect_uri");

0 commit comments

Comments
 (0)