flowable / flowable/flowable-engine

fullMessageBytes in CommentEntityImpl is platform dependent

Abierto
#3,397 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Java
Estrellas
9.5k
Forks
2.9k
Merge medio
7 h 8 min
PR fusionados (30 d)
2

Descripción

**Describe the bug**
The comment entity uses new String(byte[] bytes)/String#getBytes() to encode/decode the full message bytes, which may cause messy if there are multiple servers with different character encoding.

**Expected behavior**
The encoding and decoding should be consistent across different servers.

I suggest to use UTF8 to encode/decode, the code should be something like:
```java
public byte[] getFullMessageBytes() {
return (fullMessage != null ? fullMessage.getBytes(StandardCharsets.UTF_8) : null);
}

public void setFullMessageBytes(byte[] fullMessageBytes) {
fullMessage = (fullMessageBytes != null ? new String(fullMessageBytes, StandardCharsets.UTF_8) : null);
}
```

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza localizando CommentEntityImpl e inspecciona getFullMessageBytes y setFullMessageBytes, que actualmente dependen de la codificación predeterminada de la plataforma. Confirma que la codificación elegida se utiliza de forma coherente para ambas direcciones de conversión y, después, ejecuta las pruebas existentes relevantes del proyecto para verificar el comportamiento en distintos entornos de servidor.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
backend
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.