chore: fix openai tests and upgrade quarkus minor

l3s-learnweb · Apr 30, 2024 · 3c43939 · 3c43939
1 parent 127fc9c
commit 3c43939
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/...OpenaiConnector/src/main/java/de/l3s/interweb/connector/openai/entity/CompletionBody.java b/...OpenaiConnector/src/main/java/de/l3s/interweb/connector/openai/entity/CompletionBody.java
@@ -2,6 +2,8 @@
 
 import java.util.List;
 
+import com.fasterxml.jackson.annotation.JsonProperty;
+
 import io.quarkus.runtime.annotations.RegisterForReflection;
 
 import de.l3s.interweb.core.completion.CompletionQuery;
@@ -13,12 +15,16 @@ public final class CompletionBody {
 
     private Double temperature;
 
+    @JsonProperty("top_p")
     private Double topP;
 
+    @JsonProperty("frequency_penalty")
     private Double frequencyPenalty;
 
+    @JsonProperty("presence_penalty")
     private Double presencePenalty;
 
+    @JsonProperty("max_tokens")
     private Integer maxTokens;
 
     public CompletionBody(CompletionQuery query) {

diff --git a/...naiConnector/src/main/java/de/l3s/interweb/connector/openai/entity/CompletionMessage.java b/...naiConnector/src/main/java/de/l3s/interweb/connector/openai/entity/CompletionMessage.java
@@ -1,12 +1,15 @@
 package de.l3s.interweb.connector.openai.entity;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
+
 import io.quarkus.runtime.annotations.RegisterForReflection;
 
 import de.l3s.interweb.core.completion.Message;
 
 @RegisterForReflection
 public final class CompletionMessage {
     private String role;
+    @JsonIgnore
     private String name;
     private String content;
 

diff --git a/...b-server/src/main/java/de/l3s/interweb/server/components/auth/ApiKeyIdentityProvider.java b/...b-server/src/main/java/de/l3s/interweb/server/components/auth/ApiKeyIdentityProvider.java
@@ -25,9 +25,9 @@ public Class<ApiKeyAuthenticationRequest> getRequestType() {
     public Uni<SecurityIdentity> authenticate(ApiKeyAuthenticationRequest request, AuthenticationRequestContext authenticationRequestContext) {
         return Token.findByApiKey(request.getValue())
                 .onItem().ifNotNull()
-                .transform(consumer -> QuarkusSecurityIdentity.builder()
-                        .setPrincipal(consumer.user)
-                        .addCredential(consumer)
+                .transform(token -> QuarkusSecurityIdentity.builder()
+                        .setPrincipal(token.user)
+                        .addCredential(token)
                         .setAnonymous(false)
                         .addRole(Roles.SERVICE)
                         .build());

diff --git a/pom.xml b/pom.xml
@@ -19,7 +19,7 @@
 
         <quarkus.platform.artifact-id>quarkus-bom</quarkus.platform.artifact-id>
         <quarkus.platform.group-id>io.quarkus.platform</quarkus.platform.group-id>
-        <quarkus.platform.version>3.8.2</quarkus.platform.version>
+        <quarkus.platform.version>3.8.4</quarkus.platform.version>
 
         <surefire-plugin.version>3.2.5</surefire-plugin.version>
         <enforcer-plugin.version>3.4.1</enforcer-plugin.version>