marcel/familienarchiv
1
0
Fork 0
Code Issues 125 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

feat(search): raise NL search rate limit from 5 to 20 req/min

Browse Source 
The rule-based NLP service is <100ms vs Ollama's ~15s, making the old
limit too restrictive for normal interactive use.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-06-07 16:06:04 +02:00
committed by marcel
parent 503ed6adef
commit 08c11e567c
3 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
backend/src/main/java/org/raddatz/familienarchiv/search/NlSearchRateLimitProperties.java
View File

@@ -8,5 +8,5 @@ import org.springframework.stereotype.Component;
@ConfigurationProperties("app.nl-search.rate-limit") @ConfigurationProperties("app.nl-search.rate-limit")
@Data @Data
public class NlSearchRateLimitProperties { public class NlSearchRateLimitProperties {
private int maxRequestsPerMinute = 5; private int maxRequestsPerMinute = 20;
} }

2
backend/src/main/resources/application.yaml
View File

@@ -135,7 +135,7 @@ app:
nl-search: nl-search:
rate-limit: rate-limit:
max-requests-per-minute: 5 max-requests-per-minute: 20
ocr: ocr:
sender-model: sender-model:

6
backend/src/test/java/org/raddatz/familienarchiv/search/NlSearchControllerTest.java
View File

@@ -138,14 +138,14 @@ class NlSearchControllerTest {
.andExpect(jsonPath("$.code").value("SMART_SEARCH_UNAVAILABLE")); .andExpect(jsonPath("$.code").value("SMART_SEARCH_UNAVAILABLE"));
} }
// --- 7. 6th request in 1 minute → 429 (rate limit = 5/min default) --- // --- 7. 21st request in 1 minute → 429 (rate limit = 20/min default) ---
@Test @Test
@WithMockUser(username = "user@test.com", authorities = {"READ_ALL"}) @WithMockUser(username = "user@test.com", authorities = {"READ_ALL"})
void search_returns429_onSixthRequestWithinRateLimit() throws Exception { void search_returns429_on21stRequestWithinRateLimit() throws Exception {
when(nlQueryParserService.search(anyString(), anyString(), any())).thenReturn(makeResponse()); when(nlQueryParserService.search(anyString(), anyString(), any())).thenReturn(makeResponse());
for (int i = 0; i < 5; i++) { for (int i = 0; i < 20; i++) {
mockMvc.perform(post("/api/search/nl").with(csrf()) mockMvc.perform(post("/api/search/nl").with(csrf())
.contentType(MediaType.APPLICATION_JSON) .contentType(MediaType.APPLICATION_JSON)
.content("{\"query\":\"Briefe von Walter\",\"lang\":\"de\"}")) .content("{\"query\":\"Briefe von Walter\",\"lang\":\"de\"}"))