mirror of https://github.com/usbharu/Hideout.git
fix: Acceptヘッダーの判定方法を修正
This commit is contained in:
parent
1f4df77f98
commit
8aab1e5907
|
@ -33,13 +33,12 @@ fun Routing.usersAP(activityPubUserService: ActivityPubUserService, userService:
|
|||
class ContentTypeRouteSelector(private vararg val contentType: ContentType) : RouteSelector() {
|
||||
override fun evaluate(context: RoutingResolveContext, segmentIndex: Int): RouteSelectorEvaluation {
|
||||
|
||||
context.call.application.log.debug("Accept: ${context.call.request.accept()}")
|
||||
val requestContentType =
|
||||
ContentType.parse(context.call.request.accept() ?: return RouteSelectorEvaluation.FailedParameter)
|
||||
context.call.application.log.debug("Content-Type: {}", requestContentType)
|
||||
return if (contentType.any { contentType ->
|
||||
context.call.application.log.debug(contentType.toString())
|
||||
contentType.match(requestContentType)
|
||||
}) {
|
||||
|
||||
return if (contentType.find { contentType: ContentType -> contentType.match(requestContentType) }
|
||||
.let { true }) {
|
||||
RouteSelectorEvaluation.Constant
|
||||
} else {
|
||||
RouteSelectorEvaluation.FailedParameter
|
||||
|
|
|
@ -22,7 +22,6 @@ import io.ktor.client.statement.*
|
|||
import io.ktor.http.*
|
||||
import io.ktor.server.config.*
|
||||
import io.ktor.server.testing.*
|
||||
import org.junit.jupiter.api.Disabled
|
||||
import org.junit.jupiter.api.Test
|
||||
import org.mockito.ArgumentMatchers.anyString
|
||||
import org.mockito.kotlin.doReturn
|
||||
|
@ -99,7 +98,7 @@ class UsersAPTest {
|
|||
}
|
||||
|
||||
@Test()
|
||||
@Disabled
|
||||
// @Disabled
|
||||
fun `ユーザのURLにAcceptヘッダーをActivityとJson-LDにしてアクセスしたときPersonが返ってくる`() = testApplication {
|
||||
environment {
|
||||
config = ApplicationConfig("empty.conf")
|
||||
|
@ -166,10 +165,10 @@ class UsersAPTest {
|
|||
}
|
||||
|
||||
@Test
|
||||
@Disabled
|
||||
// @Disabled
|
||||
fun contentType_Test() {
|
||||
val listOf = listOf(ContentType.Application.JsonLd, ContentType.Application.Activity)
|
||||
assertTrue(listOf.any { contentType -> contentType.match("application/ld+json; profile=\"\\\"https://www.w3.org/ns/activitystreams\\\",application/activity+json\"") })
|
||||
assertTrue(listOf.find { contentType -> contentType.match("application/ld+json; profile=\"\\\"https://www.w3.org/ns/activitystreams\\\",application/activity+json\"") }.let { true })
|
||||
assertTrue(ContentType.Application.JsonLd.match("application/ld+json; profile=\"https://www.w3.org/ns/activitystreams\""))
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue