Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Decoding a decimal column changes the caller's Decimal.Context and never restores it

Abierto Apto para principiantes
  • ¿Algo no está bien?
#191 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
elixir
Área
backend, databases

Línea de trabajo

Lee lib/tds/types.ex y céntrate en decode_decimal/3, encode_decimal_type/1, encode_float_type/1, encode_decimal_descriptor/1, encode_data/3 y set_decimal_precision/1. El informe indica que estas llamadas modifican Decimal.Context sin usar la precisión modificada para las operaciones aritméticas; revisa las pruebas existentes de decodificación y codificación de decimales y luego verifica que el contexto del llamador permanezca sin cambios después de esos recorridos.

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

Descripción

Summary

Tds.Types.decode_decimal/3 calls Decimal.Context.set/1 with the column's precision for every decimal value it decodes, and the decimal/float encode paths set precision 38 for every decimal parameter they encode. Nothing restores the previous context. Decoding runs in the client process, so after any query that returns a decimal column, the caller's Decimal.Context precision is that column's precision instead of the default, for the rest of the process's life.

tds 2.3.8 (latest), decimal 3.2.0, Elixir 1.20.3, Ecto 3.14 / ecto_sql 3.14 with Ecto.Adapters.Tds, SQL Server 2019.

Reproduction

Decimal.Context.set(%Decimal.Context{})
Decimal.Context.get().precision
#=> 34 (decimal 3.2.0 default; 28 on older versions)

# any table with a numeric(15, 9) column
{:ok, _} = MyRepo.query("select top 1 MUCOFA from MITAUN", [])
Decimal.Context.get().precision
#=> 15

# after decoding a numeric(5, x) column the precision is 5, and then
Decimal.round(Decimal.new("1000.000000000"), 2)
#=> ** (Decimal.Error) invalid_operation: result has more digits than the precision

Where

lib/tds/types.ex:

  • decode_decimal/3 calls set_decimal_precision(precision) and then only Decimal.new/3, which does not read the context, so the call has no effect on the decoded value and only leaks.
  • encode_decimal_type/1, encode_float_type/1, encode_decimal_descriptor/1 and encode_data/3 for decimaln call set_decimal_precision(38), and then only Decimal.abs/1, Decimal.to_string/2 and Decimal.new/1, none of which round to the context precision either.
  • set_decimal_precision/1 is Decimal.Context.get() |> Map.put(:precision, p) |> Decimal.Context.set().

Impact

Every Decimal.mult/2, add/2 or div/2 the application does after a query is rounded to the last decoded column's precision instead of its own. This was silent for a long time because Decimal.round/3 ignored the context precision until decimal 3.2.0 (2026-10-10), which made round/3 follow the spec and raise, so an application that rounds a decoded value to two places now fails on a value as plain as 1000.000000000. We found it when a production import stopped completely after upgrading decimal.

Suggested fix

Drop set_decimal_precision/1 altogether: none of the five call sites does arithmetic that depends on the context precision. Should a wider precision ever be needed, Decimal.Context.with/2 restores the previous context afterwards:

Decimal.Context.with(%{Decimal.Context.get() | precision: 38}, fn -> ... end)

Happy to send a PR if that direction is acceptable.

Lenguaje dominante
Elixir
Estrellas
119
Forks
87
Sin PR fusionados en 30 d

Preparar el entorno

  • Incluye un Dockerfile o un archivo de Docker Compose
  • Sin plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de elixir-ecto/tds

Todos los issues de elixir-ecto/tds

Issues similares

Más issues de Elixir

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.