open-code-review

Projetos que seguem as melhores práticas abaixo podem se autocertificar voluntariamente e mostrar que alcançaram um selo de melhores práticas da Open Source Security Foundation (OpenSSF).

Não existe um conjunto de práticas que possa garantir que o software nunca terá defeitos ou vulnerabilidades; mesmo métodos formais podem falhar se as especificações ou suposições estiverem erradas. Nem existe qualquer conjunto de práticas que possa garantir que um projeto sustentará uma comunidade de desenvolvimento saudável e bem-funcionada. No entanto, seguir as melhores práticas pode ajudar a melhorar os resultados dos projetos. Por exemplo, algumas práticas permitem revisão multipessoal antes do lançamento, o que pode ajudar a encontrar vulnerabilidades técnicas difíceis de encontrar e ajudar a construir confiança e desejo de interação repetida entre desenvolvedores de diferentes empresas. Para ganhar um selo, todos os critérios DEVE e NÃO DEVE devem ser atendidos, todos os critérios DEVERIA devem ser atendidos OU não atendidos com justificativa, e todos os critérios SUGERIDO devem ser atendidos OU não atendidos (queremos que sejam considerados pelo menos). Se você quiser inserir texto de justificativa como um comentário genérico, em vez de ser uma justificativa de que a situação é aceitável, inicie o bloco de texto com '//' seguido de um espaço. Feedback é bem-vindo via site do GitHub como questões ou pull requests Há também uma lista de discussão para discussão geral.

Fornecemos com prazer as informações em vários idiomas, no entanto, se houver qualquer conflito ou inconsistência entre as traduções, a versão em inglês é a versão autoritativa.
Se este é o seu projeto, por favor mostre o status do seu selo na página do seu projeto! O status do selo se parece com isto: O nível do selo para o projeto 13328 é gold Aqui está como incorporá-lo:
Você pode mostrar o status do seu selo incorporando isto no seu arquivo markdown:
[![OpenSSF Best Practices](https://www.bestpractices.dev/projects/13328/badge)](https://www.bestpractices.dev/projects/13328)
ou incorporando isto no seu HTML:
<a href="https://www.bestpractices.dev/projects/13328"><img src="https://www.bestpractices.dev/projects/13328/badge"></a>


Estes são os critérios de nível Ouro. Você também pode visualizar os critérios de nível Aprovação ou Prata.

Baseline Series: Nível Básico 1 Nível Básico 2 Nível Básico 3

        

 Fundamentos 5/5 ●

  • Geral

    Observe que outros projetos podem usar o mesmo nome.

    Open-source & free — Battle-tested at Alibaba's scale. Hybrid architecture code review tool: deterministic pipelines + LLM Agent, precise line-level comments, built-in fine-tuned ruleset (NPE, thread-safety, XSS, SQL injection), OpenAI & Anthropic compatible.

    Use o formato de expressão de licença SPDX; exemplos incluem "Apache-2.0", "BSD-2-Clause", "BSD-3-Clause", "GPL-2.0+", "LGPL-3.0+", "MIT" e "(BSD-2-Clause OR Ruby)". Não inclua aspas simples ou aspas duplas.
    Se houver mais de uma linguagem, liste-as como valores separados por vírgula (espaços opcionais) e ordene-as da mais usada para a menos usada. Se houver uma longa lista, liste pelo menos as três primeiras mais comuns. Se não houver linguagem (por exemplo, este é um projeto apenas de documentação ou apenas de teste), use o caractere único "-". Use uma capitalização convencional para cada linguagem, por exemplo, "JavaScript".
    O Common Platform Enumeration (CPE) é um esquema de nomenclatura estruturado para sistemas de tecnologia da informação, software e pacotes. Ele é usado em vários sistemas e bancos de dados ao relatar vulnerabilidades.

  • Pré-requisitos


    Suficiente para um badge!

    O projeto DEVE alcançar um selo de nível prata. [achieve_silver]

  • Supervisão do projeto


    Suficiente para um badge!

    O projeto DEVE ter um "fator ônibus" de 2 ou mais. (URL obrigatória) [bus_factor]
    Um "bus factor" (também conhecido como "truck factor") é o número mínimo de membros do projeto que precisam desaparecer repentinamente de um projeto ("ser atropelados por um ônibus") antes que o projeto pare devido à falta de pessoal conhecedor ou competente. A ferramenta truck-factor pode estimar isso para projetos no GitHub. Para mais informações, consulte Assessing the Bus Factor of Git Repositories de Cosentino et al.

    The project has 10+ contributors. Multiple team members within the Alibaba organization have deep familiarity with the codebase and can maintain the project independently.
    https://github.com/alibaba/open-code-review/blob/main/GOVERNANCE.md#continuity


    Suficiente para um badge!

    O projeto DEVE ter pelo menos dois contribuidores significativos não associados. (URL obrigatória) [contributors_unassociated]
    Os contribuidores são associados se forem pagos para trabalhar pela mesma organização (como empregado ou contratado) e a organização se beneficia dos resultados do projeto. Concessões financeiras não contam como sendo da mesma organização se passarem por outras organizações (por exemplo, concessões científicas pagas a diferentes organizações de uma fonte governamental ou ONG comum não fazem com que os contribuidores sejam associados). Alguém é um contribuidor significativo se fez contribuições não triviais para o projeto no último ano. Exemplos de bons indicadores de um contribuidor significativo são: escreveu pelo menos 1.000 linhas de código, contribuiu com 50 commits, ou contribuiu com pelo menos 20 páginas de documentação.

    https://github.com/alibaba/open-code-review/graphs/contributors，The project has multiple significant contributors from unassociated organizations. Besides the lead maintainer (Alibaba), regular external contributors each have 20+ commits from non-Alibaba domains.


  • Outro


    Suficiente para um badge!

    O projeto DEVE incluir uma declaração de licença em cada arquivo-fonte. Isso PODE ser feito incluindo o seguinte dentro de um comentário perto do início de cada arquivo: SPDX-License-Identifier: [expressão de licença SPDX para o projeto]. [license_per_file]
    Isso PODE também ser feito incluindo uma declaração em linguagem natural identificando a licença. O projeto PODE também incluir uma URL estável apontando para o texto da licença, ou o texto completo da licença. Observe que o critério license_location requer que a licença do projeto esteja em um local padrão. Veja este tutorial SPDX para mais informações sobre expressões de licença SPDX. Observe a relação com copyright_per_file, cujo conteúdo normalmente precederia as informações de licença.

    Every source file includes "SPDX-License-Identifier: Apache-2.0" near the
    beginning. This is enforced by CI via make license-check (runs
    scripts/verify-license.sh). New files are automatically stamped with
    make license-add.


 Controle de Mudanças 4/4 ●

  • Repositório de código-fonte público controlado por versão


    Suficiente para um badge!

    O repositório de código do projeto DEVE usar um software de controle de versão distribuído comum (por exemplo, git ou mercurial). [repo_distributed]
    O Git não é especificamente exigido e os projetos podem usar software de controle de versão centralizado (como subversion) com justificativa.

    Repository on GitHub, which uses git. git is distributed.


    Suficiente para um badge!

    O projeto DEVE identificar claramente pequenas tarefas que podem ser realizadas por novos colaboradores ou colaboradores casuais. (URL obrigatória) [small_tasks]
    Esta identificação é tipicamente feita marcando problemas selecionados em um rastreador de problemas com uma ou mais tags que o projeto usa para o propósito, por exemplo, up-for-grabs, first-timers-only, "Small fix", microtask ou IdealFirstBug. Essas novas tarefas não precisam envolver a adição de funcionalidade; elas podem ser melhorar a documentação, adicionar casos de teste ou qualquer outra coisa que ajude o projeto e ajude o colaborador a entender mais sobre o projeto.

    CONTRIBUTING.md directs newcomers to small, well-scoped tasks via the "good first issue" and "help wanted" labels, with direct links. Available tasks: https://github.com/alibaba/open-code-review/issues?q=state%3Aopen%20label%3A%22good%20first%20issue%22


    Suficiente para um badge!

    O projeto DEVE exigir autenticação de dois fatores (2FA) para desenvolvedores para alterar um repositório central ou acessar dados sensíveis (como relatórios de vulnerabilidade privados). Este mecanismo 2FA PODE usar mecanismos sem mecanismos criptográficos, como SMS, embora isso não seja recomendado. [require_2FA]

    GitHub requires 2FA as of March 2023. [osps_ac_01_01]


    Suficiente para um badge!

    A autenticação de dois fatores (2FA) do projeto DEVERIA usar mecanismos criptográficos para evitar personificação. A autenticação 2FA baseada em Short Message Service (SMS), por si só, NÃO atende a este critério, pois não é criptografada. [secure_2FA]
    Um mecanismo de 2FA que atende a este critério seria um aplicativo de Time-based One-Time Password (TOTP) que gera automaticamente um código de autenticação que muda após um determinado período de tempo. Observe que o GitHub suporta TOTP.

    Project maintainers protect their GitHub accounts with cryptographic 2FA — TOTP authenticator apps and/or WebAuthn security keys, not SMS. This satisfies the requirement for phishing-resistant, non-SMS second factors.


 Qualidade 7/7 ●

  • Padrões de codificação


    Suficiente para um badge!

    O projeto DEVE documentar seus requisitos de revisão de código, incluindo como a revisão de código é conduzida, o que deve ser verificado e o que é necessário para ser aceitável. (URL obrigatória) [code_review_standards]
    Veja também two_person_review e contribution_requirements.

    Review and merge expectations are documented in GOVERNANCE.md ("Reviews and Merge Expectations") and CONTRIBUTING.md. https://github.com/alibaba/open-code-review/blob/main/GOVERNANCE.md


    Suficiente para um badge!

    O projeto DEVE ter pelo menos 50% de todas as modificações propostas revisadas antes do lançamento por uma pessoa diferente do autor, para determinar se é uma modificação que vale a pena e livre de problemas conhecidos que argumentariam contra sua inclusão [two_person_review]

    All changes land on the main branch exclusively through GitHub Pull Requests. Branch protection on main requires at least one approving review from someone other than the author before merge, and this rule is enforced for administrators as well (enforce_admins enabled). CI (the test job: build, race-enabled tests, coverage gate, smoke test) is a required status check for merge. Merged PRs: https://github.com/alibaba/open-code-review/pulls?q=is%3Apr+is%3Amerged


  • Sistema de compilação funcional


    Suficiente para um badge!

    O projeto DEVE ter uma compilação reproduzível. Se nenhuma compilação ocorrer (por exemplo, linguagens de script onde o código fonte é usado diretamente em vez de ser compilado), selecione "não aplicável" (N/A). (URL obrigatória) [build_reproducible]
    Uma compilação reproduzível significa que várias partes podem refazer independentemente o processo de geração de informações a partir de arquivos fonte e obter exatamente o mesmo resultado bit a bit. Em alguns casos, isso pode ser resolvido forçando algum tipo de ordenação. Desenvolvedores JavaScript podem considerar usar npm shrinkwrap e webpack OccurrenceOrderPlugin. Usuários de GCC e clang podem achar útil a opção -frandom-seed. O ambiente de compilação (incluindo o conjunto de ferramentas) pode frequentemente ser definido para partes externas especificando o hash criptográfico de um contêiner específico ou máquina virtual que eles podem usar para recompilar. O projeto de compilações reproduzíveis tem documentação sobre como fazer isso.

    The build is reproducible. All dependencies are pinned by cryptographic hash in go.sum; the build toolchain is pinned to a specific container image (golang:1.26.5) in CI; release binaries are built with CGO_ENABLED=0 for static, deterministic output; and SHA-256 checksums are generated for every release artifact via make sha256sum. See https://github.com/alibaba/open-code-review/blob/main/Makefile and https://github.com/alibaba/open-code-review/blob/main/.github/workflows/ci.yml


  • Conjunto de testes automatizados


    Suficiente para um badge!

    Um conjunto de testes DEVE ser invocável de forma padrão para aquela linguagem. (URL obrigatória) [test_invocation]
    Por exemplo, "make check", "mvn test", ou "rake test" (Ruby).

    The test suite is invoked in the standard Go way: go test ./..., wrapped as make test (which runs go test -race -count=1 across all packages). It runs automatically in CI on every push and pull request. See https://github.com/alibaba/open-code-review/blob/main/Makefile and https://github.com/alibaba/open-code-review/blob/main/.github/workflows/ci.yml


    Suficiente para um badge!

    O projeto DEVE implementar integração contínua, onde o código novo ou alterado é frequentemente integrado em um repositório de código central e testes automatizados são executados no resultado. (URL obrigatória) [test_continuous_integration]
    Na maioria dos casos, isso significa que cada desenvolvedor que trabalha em tempo integral no projeto integra pelo menos diariamente.

    https://github.com/alibaba/open-code-review/blob/main/.github/workflows/ci.yml


    Suficiente para um badge!

    O projeto DEVE ter conjunto(s) de testes automatizados FLOSS que fornecem pelo menos 90% de cobertura de instrução se houver pelo menos uma ferramenta FLOSS que possa medir este critério na linguagem selecionada. [test_statement_coverage90]

    The project is written in Go (see go.mod). Go's built-in go test -cover is a
    FLOSS statement-coverage tool, so this criterion's precondition is satisfied and
    it is reported as Met (not N/A).

    The authoritative measurement is the CI gate, which runs go test -race ./...
    over the whole module and currently reports total statement coverage of 90.2%.
    This is enforced in CI: the "Check coverage threshold" step in
    .github/workflows/ci.yml fails the build if coverage drops below 90%.

    The Makefile make coverage target is a local convenience that reports a
    slightly higher number (90.6%) because it measures a smaller package set: it
    omits -race and excludes extensions/ via PACKAGES. The two figures cover
    different package sets and are not the same measurement; the CI figure governs.

    Reproduce (authoritative, CI scope):
    go test -count=1 -race -coverprofile=coverage.out ./...
    go tool cover -func=coverage.out | grep total:

    local convenience (excludes extensions/, no -race):

    make coverage


    Suficiente para um badge!

    O projeto DEVE ter conjunto(s) de testes automatizados FLOSS que fornecem pelo menos 80% de cobertura de ramos se houver pelo menos uma ferramenta FLOSS que possa medir este critério na linguagem selecionada. [test_branch_coverage80]

    The project is written in Go (see go.mod). The Go toolchain's built-in coverage
    support (go test -cover) only offers three modes -- set, count, and atomic --
    all of which measure statement coverage. It does not measure branch (or
    condition) coverage; there is no -covermode=branch and go tool cover exposes
    no branch-coverage output. This was verified against the go1.26 toolchain via
    go help testflag and go tool cover -h. Per this criterion's own precondition
    ("if there is at least one FLOSS tool that can implement this criterion in the
    selected language"), the criterion is Not Applicable for a Go project.


 Segurança 5/5 ●

  • Usar práticas criptográficas boas e básicas

    Observe que alguns softwares não precisam usar mecanismos criptográficos. Se o seu projeto produzir software que (1) inclui, ativa ou habilita funcionalidade de criptografia, e (2) pode ser liberado dos Estados Unidos (EUA) para fora dos EUA ou para um não cidadão dos EUA, você pode ser legalmente obrigado a tomar algumas etapas extras. Normalmente isso envolve apenas o envio de um e-mail. Para mais informações, consulte a seção de criptografia de Understanding Open Source Technology & US Export Controls.

    Suficiente para um badge!

    O software produzido pelo projeto DEVE suportar protocolos seguros para todas as suas comunicações de rede, como SSHv2 ou posterior, TLS1.2 ou posterior (HTTPS), IPsec, SFTP e SNMPv3. Protocolos inseguros como FTP, HTTP, telnet, SSLv3 ou anterior, e SSHv1 DEVEM estar desabilitados por padrão, e apenas habilitados se o usuário configurá-lo especificamente. Se o software produzido pelo projeto não suportar comunicações de rede, selecione "não aplicável" (N/A). [crypto_used_network]

    Justification: All network communications use HTTPS exclusively. The project communicates with LLM API providers (OpenAI, Anthropic, etc.) via HTTPS using Go's net/http standard library. There is no support for insecure HTTP connections to API endpoints. The viewer component also serves over HTTPS when configured.


    Suficiente para um badge!

    O software produzido pelo projeto DEVE, se suportar ou usar TLS, suportar pelo menos a versão 1.2 do TLS. Observe que o predecessor do TLS foi chamado SSL. Se o software não usar TLS, selecione "não aplicável" (N/A). [crypto_tls12]

    Justification: Go's crypto/tls package enforces TLS 1.2 as the minimum version by default (since Go 1.18). The project does not override this default. All outbound HTTPS connections to LLM APIs use TLS 1.2 or later.


  • Entrega protegida contra ataques man-in-the-middle (MITM)


    Suficiente para um badge!

    O site do projeto, repositório (se acessível via web) e site de download (se separado) DEVEM incluir cabeçalhos de fortalecimento chave com valores não permissivos. (URL obrigatória) [hardened_site]
    Observe que o GitHub e o GitLab são conhecidos por atender a isso. Sites como https://securityheaders.com/ podem verificar isso rapidamente. Os principais cabeçalhos de proteção são: Content Security Policy (CSP), HTTP Strict Transport Security (HSTS), X-Content-Type-Options (como "nosniff") e X-Frame-Options. Sites web totalmente estáticos sem capacidade de fazer login por meio das páginas web poderiam omitir alguns cabeçalhos de proteção com menos risco, mas não há maneira confiável de detectar tais sites, portanto exigimos esses cabeçalhos mesmo se forem sites totalmente estáticos.

    The repository (github.com/alibaba/open-code-review) and download site
    (GitHub Releases) are hosted on GitHub, which is known to send hardened
    headers. The project website https://open-codereview.ai/ returns hardening
    headers with non-permissive values on every response: Strict-Transport-Security,
    a strict Content-Security-Policy (default-src 'self', no unsafe script-src),
    X-Content-Type-Options: nosniff, X-Frame-Options: DENY, Referrer-Policy: no-referrer,
    and Permissions-Policy. Scanning https://open-codereview.ai/ on securityheaders.com
    yields an A grade.


  • Outras questões de segurança


    Suficiente para um badge!

    O projeto DEVE ter realizado uma revisão de segurança nos últimos 5 anos. Esta revisão DEVE considerar os requisitos de segurança e o limite de segurança. [security_review]
    Isso PODE ser feito pelos membros do projeto e/ou uma avaliação independente. Esta avaliação PODE ser apoiada por ferramentas de análise estática e dinâmica, mas também deve haver revisão humana para identificar problemas (particularmente no projeto) que as ferramentas não conseguem detectar.

    A security review was performed by project members and is documented in ASSURANCE_CASE.md. It covers security requirements, the threat model, trust boundaries, Saltzer & Schroeder secure-design principles, and countermeasures mapped to OWASP Top 10 / CWE Top 25. The review includes human analysis of design-level issues, supported by static and dynamic tooling (go vet, govulncheck, the Go race detector). https://github.com/alibaba/open-code-review/blob/main/ASSURANCE_CASE.md


    Suficiente para um badge!

    Mecanismos de proteção DEVEM ser usados no software produzido pelo projeto para que defeitos de software tenham menos probabilidade de resultar em vulnerabilidades de segurança. (URL obrigatória) [hardening]
    Os mecanismos de proteção podem incluir cabeçalhos HTTP como Content Security Policy (CSP), flags de compilador para mitigar ataques (como -fstack-protector), ou flags de compilador para eliminar comportamento indefinido. Para nossos propósitos, o privilégio mínimo não é considerado um mecanismo de proteção (privilégio mínimo é importante, mas separado).

    OCR applies multiple hardening mechanisms in the software it produces:
    (1) written in Go (memory-safe) and built with CGO_ENABLED=0, eliminating
    C-family memory-corruption and undefined-behavior classes;
    (2) the local web viewer sends defense-in-depth HTTP headers on every response --
    a strict Content-Security-Policy (default-src 'self', no unsafe-inline),
    X-Content-Type-Options: nosniff, X-Frame-Options: DENY, Referrer-Policy,
    and Permissions-Policy (internal/viewer/securityheaders.go);
    (3) the viewer enforces a host-header allowlist blocking DNS-rebinding
    (internal/viewer/hostguard.go);
    (4) the Go race detector runs in CI.
    See https://github.com/alibaba/open-code-review/blob/main/ASSURANCE_CASE.md


 Análise 2/2 ●

  • Análise dinâmica de código


    Suficiente para um badge!

    O projeto DEVE aplicar pelo menos uma ferramenta de análise dinâmica a qualquer lançamento de produção proposto do software produzido pelo projeto antes de seu lançamento. [dynamic_analysis]
    Uma ferramenta de análise dinâmica examina o software executando-o com entradas específicas. Por exemplo, o projeto PODE usar uma ferramenta de fuzzing (por exemplo, American Fuzzy Lop) ou um scanner de aplicação web (por exemplo, OWASP ZAP ou w3af). Em alguns casos, o projeto OSS-Fuzz pode estar disposto a aplicar testes de fuzzing ao seu projeto. Para fins deste critério, a ferramenta de análise dinâmica precisa variar as entradas de alguma forma para procurar vários tipos de problemas ou ser um conjunto de testes automatizado com pelo menos 80% de cobertura de ramificação. A página da Wikipedia sobre análise dinâmica e a página da OWASP sobre fuzzing identificam algumas ferramentas de análise dinâmica. A(s) ferramenta(s) de análise PODEM estar focadas em procurar vulnerabilidades de segurança, mas isso não é obrigatório.

    OCR runs its full automated test suite under the Go race detector (go test -race) in CI on every push and pull request, and therefore before every release. The race detector is a dynamic-analysis tool (ThreadSanitizer-based) that instruments the running binary and detects data races and memory-ordering bugs at execution time. CI additionally runs govulncheck for reachability-based vulnerability analysis. See https://github.com/alibaba/open-code-review/blob/main/.github/workflows/ci.yml


    Suficiente para um badge!

    O projeto DEVERIA incluir muitas asserções em tempo de execução no software que produz e verificar essas asserções durante a análise dinâmica. [dynamic_analysis_enable_assertions]
    Este critério não sugere habilitar asserções durante a produção; isso é inteiramente decisão do projeto e de seus usuários. O foco deste critério é, em vez disso, melhorar a detecção de falhas durante a análise dinâmica antes da implantação. Habilitar asserções no uso em produção é completamente diferente de habilitar asserções durante a análise dinâmica (como testes). Em alguns casos, habilitar asserções no uso em produção é extremamente imprudente (especialmente em componentes de alta integridade). Existem muitos argumentos contra habilitar asserções em produção, por exemplo, bibliotecas não devem travar chamadores, sua presença pode causar rejeição por lojas de aplicativos e/ou ativar uma asserção em produção pode expor dados privados, como chaves privadas. Observe que em muitas distribuições Linux NDEBUG não é definido, então assert() em C/C++ será habilitado por padrão para produção nesses ambientes. Pode ser importante usar um mecanismo de asserção diferente ou definir NDEBUG para produção nesses ambientes.

    The produced software performs always-on runtime invariant checks: explicit panics guard violated preconditions (e.g. tool-registration invariants in internal/tool/definitions.go, embedded-config parse invariants in internal/config/allowlist), and the memory-safe Go runtime unconditionally enforces bounds, nil, and type checks. Unlike C/C++ assert(), these are never compiled out -- there is no NDEBUG-style switch that could disable them in production or testing. All are exercised during dynamic analysis: the 80+ package test suite drives the code while the race detector runs alongside, so any triggered invariant fails the build. See https://github.com/alibaba/open-code-review/blob/main/.github/workflows/ci.yml



Você pode usar ferramentas e sistemas de IA para propor alterações por meio de uma URL simples, como https://www.bestpractices.dev/pt-BR/projects/13328/choose/edit?osps_ac_01_01_status=Met&osps_ac_01_01_justification=GitHub+enforced. Veja nosso sistema de propostas de automação para saber como fazer isso. Estes dados estão disponíveis sob o Community Data License Agreement – Permissive, Version 2.0 (CDLA-Permissive-2.0). Isso significa que um Destinatário de Dados pode compartilhar os Dados, com ou sem modificações, desde que o Destinatário de Dados disponibilize o texto deste acordo com os Dados compartilhados. Por favor, dê crédito a lizhengfeng e aos contribuidores do selo de melhores práticas OpenSSF.

Entrada de selo do projeto de propriedade de: lizhengfeng.
Entrada criada em 2026-06-22 06:51:21 UTC, última atualização em 2026-08-06 04:57:39 UTC. Selo de aprovação perdido pela última vez em 2026-06-26 12:15:08 UTC. Selo de aprovação alcançado pela última vez em 2026-06-26 12:41:47 UTC.