diff --git a/src/main/java/net/javahippie/fitpub/controller/ActivityPubController.java b/src/main/java/net/javahippie/fitpub/controller/ActivityPubController.java index 8369cce..861b3c8 100644 --- a/src/main/java/net/javahippie/fitpub/controller/ActivityPubController.java +++ b/src/main/java/net/javahippie/fitpub/controller/ActivityPubController.java @@ -291,52 +291,64 @@ public class ActivityPubController { } /** - * Format activity content for ActivityPub. - * Uses plain text with Unicode symbols for maximum compatibility. + * Format activity content as HTML for ActivityPub. + * Mastodon and most Fediverse software expect HTML in the content field. */ private String formatActivityContent(Activity activity) { StringBuilder content = new StringBuilder(); - // Title (if present) + // Title if (activity.getTitle() != null && !activity.getTitle().isEmpty()) { - content.append(activity.getTitle()).append("\n\n"); + content.append("
").append(escapeHtml(activity.getTitle())).append("
"); } - // Description (if present) + // Description if (activity.getDescription() != null && !activity.getDescription().isEmpty()) { - content.append(activity.getDescription()).append("\n\n"); + content.append("").append(escapeHtml(activity.getDescription())).append("
"); } // Activity type with emoji String activityEmoji = getActivityEmoji(activity.getActivityType()); String formattedType = ActivityFormatter.formatActivityType(activity.getActivityType()); - content.append(activityEmoji).append(" ").append(formattedType); + content.append("").append(activityEmoji).append(" ").append(escapeHtml(formattedType)).append("
"); // Metrics + StringBuilder metrics = new StringBuilder(); if (activity.getTotalDistance() != null) { - content.append("\nđ ") - .append(String.format("%.2f km", activity.getTotalDistance().doubleValue() / 1000.0)); + metrics.append("đ ") + .append(String.format("%.2f km", activity.getTotalDistance().doubleValue() / 1000.0)) + .append("").append(metrics).append("
"); } return content.toString(); } + private static String escapeHtml(String text) { + if (text == null) return ""; + return text.replace("&", "&") + .replace("<", "<") + .replace(">", ">") + .replace("\"", """); + } + /** * Get emoji for activity type. */ diff --git a/src/main/java/net/javahippie/fitpub/service/ActivityPostProcessingService.java b/src/main/java/net/javahippie/fitpub/service/ActivityPostProcessingService.java index 4f89ce9..8960b5b 100644 --- a/src/main/java/net/javahippie/fitpub/service/ActivityPostProcessingService.java +++ b/src/main/java/net/javahippie/fitpub/service/ActivityPostProcessingService.java @@ -216,47 +216,33 @@ public class ActivityPostProcessingService { } /** - * Format activity content for ActivityPub Note. - * Uses plain text with Unicode symbols for maximum compatibility across Fediverse platforms. - * - * Format: - * - Title (if present) - * - Description (if present) - * - Activity type with emoji - * - Distance (if present) - * - Duration (if present) - * - Elevation gain (if present) + * Format activity content as HTML for ActivityPub Note. + * Mastodon and most Fediverse software expect HTML in the content field. * * @param activity the activity to format - * @return formatted content string + * @return formatted HTML content string */ private String formatActivityContent(Activity activity) { StringBuilder content = new StringBuilder(); - // Title (if present) if (activity.getTitle() != null && !activity.getTitle().isEmpty()) { - content.append(activity.getTitle()).append("\n\n"); + content.append("").append(escapeHtml(activity.getTitle())).append("
"); } - // Description (if present) if (activity.getDescription() != null && !activity.getDescription().isEmpty()) { - content.append(activity.getDescription()).append("\n\n"); + content.append("").append(escapeHtml(activity.getDescription())).append("
"); } - // Activity type with emoji String activityEmoji = getActivityEmoji(activity.getActivityType()); String formattedType = ActivityFormatter.formatActivityType(activity.getActivityType()); - content.append(activityEmoji).append(" ").append(formattedType); + content.append("").append(activityEmoji).append(" ").append(escapeHtml(formattedType)).append("
"); - // Metrics, each on its own line with a blank line separating from above StringBuilder metrics = new StringBuilder(); - if (activity.getTotalDistance() != null) { metrics.append("đ ") .append(String.format("%.2f km", activity.getTotalDistance().doubleValue() / 1000.0)) - .append("\n"); + .append("").append(metrics).append("
"); } return content.toString(); } + private static String escapeHtml(String text) { + if (text == null) return ""; + return text.replace("&", "&") + .replace("<", "<") + .replace(">", ">") + .replace("\"", """); + } + /** * Get an emoji for the activity type. *