Skip to content

Add OpenZeppelin, Safe, and other security offers to all-networks - #1857

Merged
eugene17kotov merged 4 commits into
Chain-Love:mainfrom
dacdoyx:add-security-openzeppelin-safe
Jul 23, 2026
Merged

Add OpenZeppelin, Safe, and other security offers to all-networks#1857
eugene17kotov merged 4 commits into
Chain-Love:mainfrom
dacdoyx:add-security-openzeppelin-safe

Conversation

@dacdoyx

@dacdoyx dacdoyx commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Adds 23 missing security offers to listings/all-networks/security.csv

Wallet: 0xde524fddf721fe3376916deff77b7e9f3593b4e9 (Ethereum)

… AuditBase, ABDK, Aegis, Polyzoa, Ziion security offers
@USS-Supervisor

Copy link
Copy Markdown
Member

Hi @dacdoyx - this PR has been inactive for more than 3 days, and required CI has not reported on the branch yet. Review should wait until required CI is green. Could you please trigger or fix CI and update the PR?

@USS-Supervisor

Copy link
Copy Markdown
Member

Hi @eugene17kotov - this stale PR looks ready for reviewer action: required CI is passing and I do not see a current creator-side blocker. I requested your review so it can keep moving. Could you please take a look when you have a chance?

@USS-Supervisor

Copy link
Copy Markdown
Member

Hi @dacdoyx, this PR is stale and review should wait because no CI checks are currently reported for the branch. Could you please update the PR so required validation runs and passes?

@USS-Supervisor

Copy link
Copy Markdown
Member

Hi @dacdoyx - closing this stale PR for now because no checks are currently reported on the add-security-openzeppelin-safe branch. The next step after my previous supervisor ping was on the PR creator side, and the PR stayed unattended for more than 7 full days with no meaningful update. Please feel free to reopen or recreate it once CI can run and pass.

@eugene17kotov

Copy link
Copy Markdown
Collaborator

@dacdoyx
Thanks for the contribution.

Only the following offer is suitable for listings/all-networks/security.csv because it is network-independent:

  • salus-penetration-testing

The following offers have network-, runtime-, or chain-specific coverage and should be moved to the relevant specific-network listings:

  • abdk
  • aegis-web3
  • audit-base-pay-as-you-go
  • audit-base-pro
  • audit-base-starter
  • audit-base-team
  • beosin-kyt
  • beosin-smart-contract-audit
  • beosin-trace
  • blockaid
  • blocksec-phalcon
  • chainsecurity
  • openzeppelin-audits
  • salus-audit
  • salus-zk-solution
  • sec3-x-ray-build
  • sec3-x-ray-enterprise
  • sec3-x-ray-essential
  • sec3-x-ray-scale
  • ziion-linux-distro

Additionally:

  • polyzoa does not have sufficient official evidence for universal network support.
  • safe does not primarily fit the security category.

@github-actions

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 0
🔗 Unique 0
✅ Successful 0
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

@github-actions

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 0
🔗 Unique 0
✅ Successful 0
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

@USS-Supervisor USS-Supervisor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE
Risk: LOW
Summary: Current diff is narrow: it moves the existing salus-penetration-testing security listing into listings/all-networks/security.csv and removes the now-duplicative specific-network rows. The canonical offer resolves to the existing Salus provider, touched CSVs parse with stable 16-column shape, no duplicate listing slugs were introduced, and no specific-network salus-penetration-testing rows remain after the move. Required validation is green on head 793ca66dde20a474c37ecbadde8d2e35133b5c5b; I refreshed check-links for this review cycle and run 29998313795 completed successfully with 0 link errors/timeouts.
Findings: None.
Confidence: HIGH

@eugene17kotov
eugene17kotov merged commit 45f025f into Chain-Love:main Jul 23, 2026
4 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.

3 participants