1
1
mirror of https://github.com/MarginaliaSearch/MarginaliaSearch.git synced 2025-10-07 03:42:39 +02:00

Compare commits

...

2 Commits

Author SHA1 Message Date
Viktor Lofgren
b4fc0c4368 (ping) Fix schema drift between prod and flyway migrations 2025-06-15 17:17:11 +02:00
Viktor Lofgren
87ee8765b8 (ping) Ensure ProtocolError->HTTP_CLIENT_ERROR retains its error message information 2025-06-15 16:54:27 +02:00
4 changed files with 6 additions and 6 deletions

View File

@@ -1,5 +1,5 @@
-- Add additional summary columns to DOMAIN_SECURITY_EVENTS table -- Add additional summary columns to DOMAIN_SECURITY_EVENTS table
-- to make it easier to make sense of certificate changes -- to make it easier to make sense of certificate changes
ALTER TABLE DOMAIN_SECURITY_EVENTS ADD COLUMN CHANGE_SCHEMA ENUM('NO_CHANGE', 'HTTP_TO_HTTPS', 'HTTPS_TO_HTTP', 'UNKNOWN') NOT NULL DEFAULT 'UNKNOWN'; ALTER TABLE DOMAIN_SECURITY_EVENTS ADD COLUMN CHANGE_SCHEMA ENUM('NONE', 'HTTP_TO_HTTPS', 'HTTPS_TO_HTTP', 'UNKNOWN') NOT NULL DEFAULT 'UNKNOWN';
OPTIMIZE TABLE DOMAIN_SECURITY_EVENTS; OPTIMIZE TABLE DOMAIN_SECURITY_EVENTS;

View File

@@ -2,11 +2,11 @@ package nu.marginalia.ping.model;
public enum SchemaChange { public enum SchemaChange {
UNKNOWN, UNKNOWN,
NO_CHANGE, NONE,
HTTP_TO_HTTPS, HTTP_TO_HTTPS,
HTTPS_TO_HTTP; HTTPS_TO_HTTP;
public boolean isSignificant() { public boolean isSignificant() {
return this != NO_CHANGE && this != UNKNOWN; return this != NONE && this != UNKNOWN;
} }
} }

View File

@@ -93,9 +93,9 @@ public record SecurityInformationChange(
SchemaChange schemaChange; SchemaChange schemaChange;
if (beforeIsHttp && afterIsHttp) { if (beforeIsHttp && afterIsHttp) {
schemaChange = SchemaChange.NO_CHANGE; schemaChange = SchemaChange.NONE;
} else if (beforeIsHttps && afterIsHttps) { } else if (beforeIsHttps && afterIsHttps) {
schemaChange = SchemaChange.NO_CHANGE; schemaChange = SchemaChange.NONE;
} else if (beforeIsHttp && afterIsHttps) { } else if (beforeIsHttp && afterIsHttps) {
schemaChange = SchemaChange.HTTP_TO_HTTPS; schemaChange = SchemaChange.HTTP_TO_HTTPS;
} else if (beforeIsHttps && afterIsHttp) { } else if (beforeIsHttps && afterIsHttp) {

View File

@@ -145,7 +145,7 @@ public class HttpPingService {
domainReference.nodeId(), domainReference.nodeId(),
oldPingStatus, oldPingStatus,
ErrorClassification.HTTP_CLIENT_ERROR, ErrorClassification.HTTP_CLIENT_ERROR,
null); rsp.errorMessage());
newSecurityInformation = null; newSecurityInformation = null;
} }
case HttpResponse httpResponse -> { case HttpResponse httpResponse -> {