Merge pull request #6700 from element-hq/feature/bma/lenientJson

Let our Json parser accept comments and trailing comma.
This commit is contained in:
Benoit Marty 2026-05-05 17:08:21 +02:00 committed by GitHub
commit 65ddc4355e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 10 additions and 2 deletions

View file

@ -23,6 +23,13 @@ fun interface JsonProvider {
@ContributesBinding(AppScope::class)
@SingleIn(AppScope::class)
class DefaultJsonProvider : JsonProvider {
private val json: Json by lazy { Json { ignoreUnknownKeys = true } }
private val json: Json by lazy {
Json {
ignoreUnknownKeys = true
allowComments = true
allowTrailingComma = true
}
}
override fun invoke() = json
}

View file

@ -80,7 +80,8 @@ class DefaultSessionWellknownRetrieverTest {
"registration_helper_url": "a_registration_url",
"enforce_element_pro": true,
"rageshake_url": "a_rageshake_url",
"other": true
// Note the trailing comma, and the comment!
"other": true,
}""".trimIndent().toByteArray()
)
},