Skip to content

Atualiza bibliotecas para otimizar URLs clássicas extensas - #146

Merged
pitangainnovare merged 2 commits into
scieloorg:mainfrom
pitangainnovare:perf/classic-url-fast-path
Sep 13, 2026
Merged

pitangainnovare merged 2 commits into
scieloorg:mainfrom
pitangainnovare:perf/classic-url-fast-path

Conversation

@pitangainnovare

Copy link
Copy Markdown
Contributor

O que esse PR faz?

Atualiza o usage para consumir scielo_usage_counter 2.2.3 e scielo_log_validator 2.0.4, incorporando a otimização do processamento de URLs clássicas extensas. A versão da aplicação passa a ser 2.4.2.

Onde a revisão poderia começar?

requirements/base.txt, onde estão as duas versões internas atualizadas, e VERSION.

Como este poderia ser testado manualmente?

Após a publicação das duas bibliotecas:

docker compose -f local.yml build django
docker compose -f local.yml run --rm django pytest -q
docker compose -f local.yml run --rm django python manage.py makemigrations --check --dry-run

A suíte completa foi executada com as bibliotecas candidatas: 238 testes passaram e 2 foram ignorados. As integrações Classic, BunnyNet e Books também passaram. Não há migrations.

Algum cenário de contexto que queira dar?

Logs clássicos com parâmetros de consulta excepcionalmente extensos provocavam retrocesso excessivo em expressões regulares do tradutor. Na amostra real de MEX, a versão anterior e a candidata geraram exatamente 52.271 documentos COUNTER, 67.224 documentos analíticos e o mesmo SHA 4ff1dfa2ceeee18040fcabdc3271cab67d3ff89e23f3b637f3599537de03c1ad. O tempo caiu de 303,45 para 23,44 segundos.

Este PR depende da publicação das bibliotecas presentes nos PRs scieloorg/scielo_usage_counter#63 e scieloorg/scielo_log_validator#24.

Screenshots

Não aplicável.

Quais são os tickets relevantes?

Referências


Segurança da informação (NSI.04)

Este PR manipula dados sensíveis ou pessoais (LGPD)?

  • Não

Este PR altera autenticação, autorização, controle de acesso ou gerenciamento de sessão?

  • Não

Este PR introduz, atualiza ou remove dependências de terceiros?

  • Não — atualiza somente duas bibliotecas internas da SciELO.

Este PR foi validado pelo pipeline de segurança (SonarQube / Trivy)?

  • Não aplicável localmente — o pipeline do PR fará a validação da imagem; o gancho local do gitleaks não encontrou segredos.

Este PR concatena, monta ou executa comandos SQL, HTML ou JavaScript a partir de entrada externa?

  • Não

Este PR expõe novos endpoints, telas ou serviços?

  • Não

Algum segredo, senha, chave ou token está sendo adicionado ao código-fonte?

  • Não, nenhum segredo foi commitado

@pitangainnovare
pitangainnovare merged commit 654ae52 into scieloorg:main Sep 13, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant