fix(multitenancy): corrige downgrade NU1605 no Softprime em restore limpo - #37
Open
cloviscoli wants to merge 2 commits into
Open
fix(multitenancy): corrige downgrade NU1605 no Softprime em restore limpo#37cloviscoli wants to merge 2 commits into
cloviscoli wants to merge 2 commits into
Conversation
…impo As refs 2.2.x do Microsoft.AspNetCore.Http saíram de lista; em restore limpo (CI) o NuGet sobe para 2.3.11, que puxa Extensions.Options 8.0.x -> DI.Abstractions 8.0.0, conflitando com a ref direta antiga (3.1.1) e falhando como erro sob TreatWarningsAsErrors. Alinha Http* em 2.3.11 e Extensions.* em 8.0.x (todos suportam netstandard2.0). Adiciona '!' em tenantIdentifier no Set (guard já garante não-nulo; netstandard2.0 não tem a anotação NotNullWhen de IsNullOrWhiteSpace). https://claude.ai/code/session_01BnjVoHxcLymD2Mp9RcdG3u
…de teste Advisory novo (NU1903) no SQLitePCLRaw.lib.e_sqlite3 2.1.11 — transitivo do SQLite in-memory usado só por EF.Tests/NH.Tests — quebrava o restore sob TreatWarningsAsErrors. Dependência de teste não vai para nenhum pacote publicado, então o advisory segue como warning visível mas não bloqueia CI/release. Os projetos publicáveis mantêm a auditoria como erro (proteção real inalterada). https://claude.ai/code/session_01BnjVoHxcLymD2Mp9RcdG3u
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problema
O CI da master quebrou com
NU1605(package downgrade) noSoftprime.Multitenancydurantedotnet restore:Causa: as versões
2.2.xdoMicrosoft.AspNetCore.Httpsaíram de lista no NuGet.org. Em restore limpo (como o do runner), o NuGet sobe para2.3.11, que arrasta transitivamenteExtensions.Options 8.0.x → DI.Abstractions 8.0.0, conflitando com a referência direta antiga (3.1.1). Sob oTreatWarningsAsErrorsligado na Fase 5, esse downgrade (normalmente warning) virou erro. Não reproduzia em builds com cache quente — só em restore limpo.Correção
Microsoft.AspNetCore.Http*em2.3.11eMicrosoft.Extensions.*em8.0.xnoSoftprime.Multitenancy.csproj(todos suportam netstandard2.0), eliminando o downgrade.Caching.Memory 8.0, expondo umCS8604emMemoryCacheTenantResolver.cs. Resolvido com!emtenantIdentifiernoSet— o guardIsNullOrWhiteSpacejá garante não-nulo; o netstandard2.0 apenas não tem a anotação[NotNullWhen(false)]que o net10 usa para inferir isso. Zero mudança de comportamento.Validação local
pack+ package validation OK nos dois pacotes.🤖 Generated with Claude Code
https://claude.ai/code/session_01BnjVoHxcLymD2Mp9RcdG3u
Generated by Claude Code